Skip to content

Commit 0f1489d

Browse files
Remove unnecessary overridden methods (#13336)
The commit removes two overridden methods that were unnecessary: the 'stop()' method in the RelaySessionFactory.java file and the 'markSupported()' method in the TeeReader.java file. These methods were not contributing to the functionality of the code, hence their removal simplifies the codebase and improves readability. Co-authored-by: Diego Molina <diemol@users.noreply.github.com>
1 parent c8429ec commit 0f1489d

File tree

2 files changed

+1
-11
lines changed

2 files changed

+1
-11
lines changed

java/src/org/openqa/selenium/grid/node/relay/RelaySessionFactory.java

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -187,12 +187,7 @@ public Either<WebDriverException, ActiveSession> apply(CreateSessionRequest sess
187187
upstream,
188188
stereotype,
189189
mergedCapabilities,
190-
Instant.now()) {
191-
@Override
192-
public void stop() {
193-
// no-op
194-
}
195-
});
190+
Instant.now()) {});
196191
} catch (Exception e) {
197192
span.setAttribute(AttributeKey.ERROR.getKey(), true);
198193
span.setStatus(Status.CANCELLED);

java/src/org/openqa/selenium/grid/web/TeeReader.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -50,9 +50,4 @@ public void close() throws IOException {
5050
sink.close();
5151
}
5252
}
53-
54-
@Override
55-
public boolean markSupported() {
56-
return false;
57-
}
5853
}

0 commit comments

Comments
 (0)