Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(563)

Unified Diff: content/public/android/java/src/org/chromium/content/browser/BindingManagerImpl.java

Issue 1178253005: Use Chromium's logging utility for content files (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/public/android/java/src/org/chromium/content/browser/BindingManagerImpl.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/BindingManagerImpl.java b/content/public/android/java/src/org/chromium/content/browser/BindingManagerImpl.java
index 728699e38e7192555aeca66f39eaa39dfa456517..b2735a1f2b5b548d5af2be1e54b8baa3e5a8f4b3 100644
--- a/content/public/android/java/src/org/chromium/content/browser/BindingManagerImpl.java
+++ b/content/public/android/java/src/org/chromium/content/browser/BindingManagerImpl.java
@@ -4,9 +4,9 @@
package org.chromium.content.browser;
-import android.util.Log;
import android.util.SparseArray;
+import org.chromium.base.Log;
import org.chromium.base.SysUtils;
import org.chromium.base.ThreadUtils;
import org.chromium.base.VisibleForTesting;
@@ -15,7 +15,7 @@ import org.chromium.base.VisibleForTesting;
* Manages oom bindings used to bound child services.
*/
class BindingManagerImpl implements BindingManager {
- private static final String TAG = "BindingManager";
+ private static final String TAG = "cr.BindingManager";
// Delay of 1 second used when removing temporary strong binding of a process (only on
// non-low-memory devices).
@@ -213,8 +213,8 @@ class BindingManagerImpl implements BindingManager {
}
if (managedConnection == null) {
- Log.w(TAG, "Cannot setInForeground() - never saw a connection for the pid: "
- + Integer.toString(pid));
+ Log.w(TAG, "Cannot setInForeground() - never saw a connection for the pid: %d",
+ pid);
return;
}
@@ -237,8 +237,8 @@ class BindingManagerImpl implements BindingManager {
}
if (managedConnection == null) {
- Log.w(TAG, "Cannot call determinedVisibility() - never saw a connection for the pid: "
- + Integer.toString(pid));
+ Log.w(TAG, "Cannot call determinedVisibility() - never saw a connection for the pid:"
+ + "%d", pid);
return;
}

Powered by Google App Engine
This is Rietveld 408576698