Skip to content

Commit 790fd61

Browse files
authored
[java] Replace anonymous types with lambdas (#15043)
1 parent f33ad97 commit 790fd61

File tree

4 files changed

+16
-34
lines changed

4 files changed

+16
-34
lines changed

java/src/org/openqa/selenium/chromium/AddHasCdp.java

Lines changed: 7 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -47,20 +47,15 @@ public Class<HasCdp> getDescribedInterface() {
4747

4848
@Override
4949
public HasCdp getImplementation(Capabilities capabilities, ExecuteMethod executeMethod) {
50-
return new HasCdp() {
51-
@Override
52-
public Map<String, Object> executeCdpCommand(
53-
String commandName, Map<String, Object> parameters) {
54-
Require.nonNull("Command name", commandName);
55-
Require.nonNull("Parameters", parameters);
50+
return (commandName, parameters) -> {
51+
Require.nonNull("Command name", commandName);
52+
Require.nonNull("Parameters", parameters);
5653

57-
Map<String, Object> toReturn =
58-
(Map<String, Object>)
59-
executeMethod.execute(
60-
EXECUTE_CDP, Map.of("cmd", commandName, "params", parameters));
54+
Map<String, Object> toReturn =
55+
(Map<String, Object>)
56+
executeMethod.execute(EXECUTE_CDP, Map.of("cmd", commandName, "params", parameters));
6157

62-
return Map.copyOf(toReturn);
63-
}
58+
return Map.copyOf(toReturn);
6459
};
6560
}
6661
}

java/src/org/openqa/selenium/chromium/AddHasLaunchApp.java

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -57,13 +57,9 @@ public Class<HasLaunchApp> getDescribedInterface() {
5757

5858
@Override
5959
public HasLaunchApp getImplementation(Capabilities capabilities, ExecuteMethod executeMethod) {
60-
return new HasLaunchApp() {
61-
@Override
62-
public void launchApp(String id) {
63-
Require.nonNull("id of Chromium App", id);
64-
65-
executeMethod.execute(LAUNCH_APP, Map.of("id", id));
66-
}
60+
return id -> {
61+
Require.nonNull("id of Chromium App", id);
62+
executeMethod.execute(LAUNCH_APP, Map.of("id", id));
6763
};
6864
}
6965
}

java/src/org/openqa/selenium/chromium/AddHasPermissions.java

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -57,15 +57,11 @@ public Class<HasPermissions> getDescribedInterface() {
5757

5858
@Override
5959
public HasPermissions getImplementation(Capabilities capabilities, ExecuteMethod executeMethod) {
60-
return new HasPermissions() {
61-
@Override
62-
public void setPermission(String name, String value) {
63-
Require.nonNull("Permission name", name);
64-
Require.nonNull("Permission value", value);
65-
66-
executeMethod.execute(
67-
SET_PERMISSION, Map.of("descriptor", Map.of("name", name), "state", value));
68-
}
60+
return (name, value) -> {
61+
Require.nonNull("Permission name", name);
62+
Require.nonNull("Permission value", value);
63+
executeMethod.execute(
64+
SET_PERMISSION, Map.of("descriptor", Map.of("name", name), "state", value));
6965
};
7066
}
7167
}

java/src/org/openqa/selenium/safari/AddHasDebugger.java

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -55,11 +55,6 @@ public Class<HasDebugger> getDescribedInterface() {
5555

5656
@Override
5757
public HasDebugger getImplementation(Capabilities capabilities, ExecuteMethod executeMethod) {
58-
return new HasDebugger() {
59-
@Override
60-
public void attachDebugger() {
61-
executeMethod.execute(ATTACH_DEBUGGER, null);
62-
}
63-
};
58+
return () -> executeMethod.execute(ATTACH_DEBUGGER, null);
6459
}
6560
}

0 commit comments

Comments
 (0)