RFR: 8344231: SecurityManager cleanup in java.lang.module and jdk.internal.module [v2]

[Eirik Bjørsnøs]

> Please review this PR which removes code depending on SecurityManager from the `java.lang.module` and `jdk.internal.module` packages:
> 
> * `ModuleFinder::ofSystem` is updated to not check the `"accessSystemModules"` RuntimePermission
> * The `ModuleReferences.ExplodedModuleReader` constructor is updated to remove the check that the caller has access to the directory.
> * `Modules::addProvides` is updated to not use `AccessController::doPrivileged` when getting the module class loader
> * `SystemModuleFinder::ofSystem` is updated to not returned a wrapped result calling `AccessController::doPrivileged`
> * `SystemModuleFinders.SystemImage` is updated to remove a class level comment referring to the SM
> * `SystemModuleFinders.SystemModuleReader::checkPermissionToConnect` is removed. The URI constructor parameter in `SystemModuleReader` becomes unused and is removed.
> 
> 
> Testing: I have run `test/jdk/java/lang/module`, `test/jdk/java/lang/ModuleTests`,  and `test/jdk/java/lang/ModuleLayer` successfully. GHA results pending.

Eirik Bjørsnøs has updated the pull request incrementally with two additional commits since the last revision:

 - Remove empty line in addProvides
 - Add full stop after class description for SystemImage

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/22113/files
  - new: https://git.openjdk.org/jdk/pull/22113/files/2fcec8fc..7d53f122

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=22113&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=22113&range=00-01

  Stats: 2 lines in 2 files changed: 0 ins; 1 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/22113.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/22113/head:pull/22113

PR: https://git.openjdk.org/jdk/pull/22113