Cherry pick PR #2108: Add log API for non release builds. (#2112)

Refer to the original PR: youtube/cobalt#2108 Make v(),d(),i() logs not print for release builds. These logs still prints for fishfood and dogfood builds. b/315852240 Co-authored-by: Colin Liang <[email protected]>
youtube · Dec 16, 2023 · a70afc2 · a70afc2
1 parent 26e7778
commit a70afc2
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/starboard/android/apk/app/src/main/java/dev/cobalt/util/Log.java b/starboard/android/apk/app/src/main/java/dev/cobalt/util/Log.java
@@ -90,15 +90,24 @@ private static int logWithMethod(
   }
 
   public static int v(String tag, String messageTemplate, Object... args) {
-    return logWithMethod(logV, tag, messageTemplate, args);
+    if (android.util.Log.isLoggable(TAG, android.util.Log.VERBOSE)) {
+      return logWithMethod(logV, tag, messageTemplate, args);
+    }
+    return 0;
   }
 
   public static int d(String tag, String messageTemplate, Object... args) {
-    return logWithMethod(logD, tag, messageTemplate, args);
+    if (android.util.Log.isLoggable(TAG, android.util.Log.DEBUG)) {
+      return logWithMethod(logD, tag, messageTemplate, args);
+    }
+    return 0;
   }
 
   public static int i(String tag, String messageTemplate, Object... args) {
-    return logWithMethod(logI, tag, messageTemplate, args);
+    if (android.util.Log.isLoggable(TAG, android.util.Log.INFO)) {
+      return logWithMethod(logI, tag, messageTemplate, args);
+    }
+    return 0;
   }
 
   public static int w(String tag, String messageTemplate, Object... args) {