• Fix resource loading in OSGi environment by using resource owner class
• Deprecate single-argument resourceAsString() method
• Add new overloaded resourceAsString(Class, String) method
• Update all callers to pass resource owner class parameter

Add resource owner class parameter to resourceAsString

• Deprecated single-argument resourceAsString(String) method with javadoc warning
• Added new overloaded resourceAsString(Class<?>, String) method that accepts resource owner class
• Updated deprecated method to delegate to new method using Read.class

java/src/org/openqa/selenium/io/Read.java

Pass resource owner class to resourceAsString

• Updated domMutation() method to pass CdpEventTypes.class to resourceAsString()
• Ensures resource is loaded from correct bundle in OSGi environment

java/src/org/openqa/selenium/devtools/events/CdpEventTypes.java

Pass resource owner class to resourceAsString

• Updated addDomMutationHandler() method to pass getClass() to resourceAsString()
• Ensures bidi-mutation-listener.js is loaded from correct bundle

java/src/org/openqa/selenium/remote/RemoteScript.java

Description

The new overload resourceAsString(Class<?>, String) dereferences resourceOwner and passes
resource to getResourceAsStream without validating required inputs, which can cause an unclear
NullPointerException at runtime. This violates the fail-fast requirement for required inputs with
actionable errors.

Code

java/src/org/openqa/selenium/io/Read.java[R66-68]

+  public static String resourceAsString(Class<?> resourceOwner, String resource) {
+    try (InputStream stream = resourceOwner.getResourceAsStream(resource)) {
      if (stream == null) {

Evidence

PR Compliance ID 7 requires early validation of required inputs with clear errors; the added method
uses resourceOwner.getResourceAsStream(resource) without any non-null/validity checks, so
missing/invalid inputs can fail later with a non-actionable NPE.

java/src/org/openqa/selenium/io/Read.java[66-68]
Best Practice: Learned patterns

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
`Read.resourceAsString(Class<?>, String)` does not validate required inputs (`resourceOwner`, `resource`) before use, which can lead to runtime `NullPointerException` rather than a clear, actionable error.

## Issue Context
This overload is newly added and is now used by multiple call sites; it should fail fast with a clear message when called incorrectly.

## Fix Focus Areas
- java/src/org/openqa/selenium/io/Read.java[66-70]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

Description

ReadTest loads a remote-driver JS resource using Read.class as the resource owner, which is the same
classloader mismatch this PR is meant to fix in OSGi. This test will not validate the new API’s
cross-bundle behavior and would still fail when selenium-io and selenium-remote-driver are separate
bundles.

Code

java/test/org/openqa/selenium/io/ReadTest.java[R45-46]

+    String script = Read.resourceAsString(Read.class, "/org/openqa/selenium/remote/isDisplayed.js");
    assertThat(script).isNotBlank().contains("function(){");

Evidence

Read.class is in the selenium-io module (java/src/org/openqa/selenium/io/BUILD.bazel defines
java_library(name="io")), while isDisplayed.js is packaged as a resource of selenium-remote-driver
(java/src/org/openqa/selenium/remote/BUILD.bazel has resources including ":is-displayed" with out =
"isDisplayed.js"). In OSGi these are different bundles/classloaders, so using Read.class as the
owner would not find /org/openqa/selenium/remote/isDisplayed.js.

java/test/org/openqa/selenium/io/ReadTest.java[43-47]
java/src/org/openqa/selenium/io/BUILD.bazel[1-17]
java/src/org/openqa/selenium/remote/BUILD.bazel[48-56]
java/src/org/openqa/selenium/remote/BUILD.bazel[91-101]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### Issue description
`ReadTest` uses `Read.class` as the `resourceOwner` when loading `/org/openqa/selenium/remote/isDisplayed.js`. In OSGi this will still fail because `Read` (selenium-io bundle) can’t see resources packaged in the selenium-remote-driver bundle.

### Issue Context
The PR adds `Read.resourceAsString(Class<?> resourceOwner, String resource)` specifically to select the correct bundle classloader. The test should therefore use a class that lives in the same module/bundle that owns `/org/openqa/selenium/remote/isDisplayed.js`.

### Fix Focus Areas
- java/test/org/openqa/selenium/io/ReadTest.java[43-47]
- (if needed) java/test/org/openqa/selenium/io/BUILD.bazel[4-14] to add a dependency on a module containing an appropriate owner class

### Suggested change
Replace `Read.class` with a class from the `org.openqa.selenium.remote` module (e.g., `org.openqa.selenium.remote.codec.w3c.W3CHttpCommandCodec.class`), and add the necessary test dependency if compilation requires it.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools