[java] use --enable-chrome-logs to redirect browser I/O streams

Author: Delta456

java/src/org/openqa/selenium/chrome/ChromeDriverService.java

@@ -292,7 +292,6 @@ protected void loadSystemProperties() {
     protected List<String> createArgs() {
       List<String> args = new ArrayList<>();
       args.add(String.format(Locale.ROOT, "--port=%d", getPort()));
-
       // Readable timestamp and append logs only work if log path is specified in args
       // Cannot use logOutput because goog:loggingPrefs requires --log-path get sent
       if (getLogFile() != null) {
@@ -316,6 +315,8 @@ protected List<String> createArgs() {
       if (Boolean.TRUE.equals(disableBuildCheck)) {
         args.add("--disable-build-check");
       }
+      // Suppress or redirect browser I/O Streams
+      args.add("--enable-chrome-logs");
 
       return unmodifiableList(args);
     }

java/test/org/openqa/selenium/chrome/ChromeDriverServiceTest.java

@@ -61,19 +61,21 @@ void testScoring() {
   void logLevelLastWins() {
     ChromeDriverService.Builder builderMock = spy(ChromeDriverService.Builder.class);
 
-    List<String> silentLast = Arrays.asList("--port=1", "--log-level=OFF");
+    List<String> silentLast = Arrays.asList("--port=1", "--log-level=OFF", "--enable-chrome-logs");
     builderMock.withLogLevel(ChromiumDriverLogLevel.ALL).usingPort(1).withSilent(true).build();
     verify(builderMock).createDriverService(any(), anyInt(), any(), eq(silentLast), any());
 
-    List<String> silentFirst = Arrays.asList("--port=1", "--log-level=DEBUG");
+    List<String> silentFirst =
+        Arrays.asList("--port=1", "--log-level=DEBUG", "--enable-chrome-logs");
     builderMock.withSilent(true).withLogLevel(ChromiumDriverLogLevel.DEBUG).usingPort(1).build();
     verify(builderMock).createDriverService(any(), anyInt(), any(), eq(silentFirst), any());
 
-    List<String> verboseLast = Arrays.asList("--port=1", "--log-level=ALL");
+    List<String> verboseLast = Arrays.asList("--port=1", "--log-level=ALL", "--enable-chrome-logs");
     builderMock.withLogLevel(ChromiumDriverLogLevel.OFF).usingPort(1).withVerbose(true).build();
     verify(builderMock).createDriverService(any(), anyInt(), any(), eq(verboseLast), any());
 
-    List<String> verboseFirst = Arrays.asList("--port=1", "--log-level=INFO");
+    List<String> verboseFirst =
+        Arrays.asList("--port=1", "--log-level=INFO", "--enable-chrome-logs");
     builderMock.withVerbose(true).withLogLevel(ChromiumDriverLogLevel.INFO).usingPort(1).build();
     verify(builderMock).createDriverService(any(), anyInt(), any(), eq(verboseFirst), any());
   }
@@ -83,7 +85,8 @@ void logLevelLastWins() {
   void ignoreFalseLogging() {
     ChromeDriverService.Builder builderMock = spy(ChromeDriverService.Builder.class);
 
-    List<String> falseSilent = Arrays.asList("--port=1", "--log-level=DEBUG");
+    List<String> falseSilent =
+        Arrays.asList("--port=1", "--log-level=DEBUG", "--enable-chrome-logs");
     builderMock.withLogLevel(ChromiumDriverLogLevel.DEBUG).usingPort(1).withSilent(false).build();
     verify(builderMock).createDriverService(any(), anyInt(), any(), eq(falseSilent), any());
   }