Clean up some stylistic nits. No logical changes

SeleniumHQ · Sep 28, 2021 · c62dbf9 · c62dbf9
1 parent 12a14a2
commit c62dbf9
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 8 deletions.
diff --git a/java/src/org/openqa/selenium/chromium/AddHasCasting.java b/java/src/org/openqa/selenium/chromium/AddHasCasting.java
@@ -51,27 +51,33 @@ public Class<HasCasting> getDescribedInterface() {
   @Override
   public HasCasting getImplementation(Capabilities capabilities, ExecuteMethod executeMethod) {
     return new HasCasting() {
-      @Override public List<Map<String, String>> getCastSinks() {
+      @SuppressWarnings("unchecked")
+      @Override
+      public List<Map<String, String>> getCastSinks() {
         return (List<Map<String, String>>) executeMethod.execute(GET_CAST_SINKS, null);
       }
 
-      @Override public void selectCastSink(String deviceName) {
+      @Override
+      public void selectCastSink(String deviceName) {
         Require.nonNull("Device Name", deviceName);
 
         executeMethod.execute(SET_CAST_SINK_TO_USE, ImmutableMap.of("sinkName", deviceName));
       }
 
-      @Override public void startTabMirroring(String deviceName) {
+      @Override
+      public void startTabMirroring(String deviceName) {
         Require.nonNull("Device Name", deviceName);
 
         executeMethod.execute(START_CAST_TAB_MIRRORING, ImmutableMap.of("sinkName", deviceName));
       }
 
-      @Override public String getCastIssueMessage() {
+      @Override
+      public String getCastIssueMessage() {
         return executeMethod.execute(GET_CAST_ISSUE_MESSAGE, null).toString();
       }
 
-      @Override public void stopCasting(String deviceName) {
+      @Override
+      public void stopCasting(String deviceName) {
         Require.nonNull("Device Name", deviceName);
 
         executeMethod.execute(STOP_CASTING, ImmutableMap.of("sinkName", deviceName));

diff --git a/java/src/org/openqa/selenium/chromium/AddHasNetworkConditions.java b/java/src/org/openqa/selenium/chromium/AddHasNetworkConditions.java
@@ -42,8 +42,8 @@ public class AddHasNetworkConditions implements AugmenterProvider<HasNetworkCond
 
 
   private static final Map<String, CommandInfo> COMMANDS = ImmutableMap.of(
-    GET_NETWORK_CONDITIONS, new CommandInfo("/session/:sessionId/chromium/network_conditions",HttpMethod.GET),
-    SET_NETWORK_CONDITIONS, new CommandInfo("/session/:sessionId/chromium/network_conditions",HttpMethod.POST),
+    GET_NETWORK_CONDITIONS, new CommandInfo("/session/:sessionId/chromium/network_conditions", HttpMethod.GET),
+    SET_NETWORK_CONDITIONS, new CommandInfo("/session/:sessionId/chromium/network_conditions", HttpMethod.POST),
     DELETE_NETWORK_CONDITIONS, new CommandInfo("/session/:sessionId/chromium/network_conditions", HttpMethod.DELETE));
 
   @Override
@@ -66,6 +66,7 @@ public HasNetworkConditions getImplementation(Capabilities capabilities, Execute
     return new HasNetworkConditions() {
       @Override
       public ChromiumNetworkConditions getNetworkConditions() {
+        @SuppressWarnings("unchecked")
         Map<String, Object> result = (Map<String, Object>) executeMethod.execute(GET_NETWORK_CONDITIONS, null);
         ChromiumNetworkConditions networkConditions = new ChromiumNetworkConditions();
         networkConditions.setOffline((Boolean) result.getOrDefault(ChromiumNetworkConditions.OFFLINE, false));

diff --git a/java/src/org/openqa/selenium/chromium/HasCdp.java b/java/src/org/openqa/selenium/chromium/HasCdp.java
@@ -39,5 +39,5 @@ public interface HasCdp {
    * @param parameters any information needed to execute the Dev Tools command.
    * @return the name and value of the response.
    */
-  public Map<String, Object> executeCdpCommand(String commandName, Map<String, Object> parameters);
+  Map<String, Object> executeCdpCommand(String commandName, Map<String, Object> parameters);
 }