chore: only log har when pw:api is enabled (#977)

Author: yury-s

playwright/src/main/java/com/microsoft/playwright/impl/HARRouter.java

@@ -26,6 +26,7 @@
 import java.util.Base64;
 import java.util.Map;
 
+import static com.microsoft.playwright.impl.LoggingSupport.isApiLoggingEnabled;
 import static com.microsoft.playwright.impl.LoggingSupport.logApi;
 import static com.microsoft.playwright.impl.Serialization.fromNameValues;
 import static com.microsoft.playwright.impl.Serialization.gson;
@@ -66,7 +67,9 @@ void handle(Route route) {
     String action = response.get("action").getAsString();
     if ("redirect".equals(action)) {
       String redirectURL = response.get("redirectURL").getAsString();
-      logApi("HAR: " + route.request().url() + " redirected to " + redirectURL);
+      if (isApiLoggingEnabled()) {
+        logApi("HAR: " + route.request().url() + " redirected to " + redirectURL);
+      }
       ((RouteImpl) route).redirectNavigationRequest(redirectURL);
       return;
     }
@@ -83,7 +86,9 @@ void handle(Route route) {
     }
 
     if ("error".equals(action)) {
-      logApi("HAR: " + response.get("message").getAsString());
+      if (isApiLoggingEnabled()) {
+        logApi("HAR: " + response.get("message").getAsString());
+      }
       // Report the error, but fall through to the default handler.
     }
 

playwright/src/main/java/com/microsoft/playwright/impl/LoggingSupport.java

@@ -60,6 +60,10 @@ static void logWithTimestamp(String message) {
     System.err.println(timestamp + " " + message);
   }
 
+  static boolean isApiLoggingEnabled() {
+    return isEnabled;
+  }
+
   static void logApi(String message) {
     // This matches log format produced by the server.
     logWithTimestamp("pw:api " + message);

playwright/src/main/java/com/microsoft/playwright/impl/Router.java

@@ -39,9 +39,6 @@ private static class RouteInfo {
     }
 
     boolean handle(RouteImpl route) {
-      if (times != null && times <= 0) {
-        return false;
-      }
       if (!matcher.test(route.request().url())) {
         return false;
       }

playwright/src/test/java/com/microsoft/playwright/TestBrowserContextHar.java

@@ -326,8 +326,7 @@ void shouldRoundTripHarWithPostData(@TempDir Path tmpDir) {
       assertEquals("3", page2.evaluate(fetchFunction, "3"));
       assertEquals("3", page2.evaluate(fetchFunction, "3"));
       try {
-        Object result = page2.evaluate(fetchFunction, "4");
-        System.out.println(result);
+        page2.evaluate(fetchFunction, "4");
         fail("did not throw");
       } catch (PlaywrightException e) {
       }

playwright/src/test/java/com/microsoft/playwright/TestPageRequestFallback.java

@@ -133,10 +133,14 @@ void shouldFallBackAfterException() {
   @Test
   void shouldChainOnce() {
     page.route("**/empty.html", route -> {
+      System.out.println("before fulfill");
       route.fulfill(new Route.FulfillOptions().setStatus(200).setBody("fulfilled one"));
+      System.out.println("after fulfill");
     }, new Page.RouteOptions().setTimes(1));
     page.route("**/empty.html", route -> {
+      System.out.println("before fallback");
       route.fallback();
+      System.out.println("after fallback");
     }, new Page.RouteOptions().setTimes(1));
     Response response = page.navigate(server.EMPTY_PAGE);
     assertEquals("fulfilled one", response.text());