Index: content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java |
diff --git a/content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java b/content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java |
index c72fb6acd6b829f33e84d86978dae53b6b9095f6..03f312b257810c295d2f30d0fd36375f3ee39d8f 100644 |
--- a/content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java |
+++ b/content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java |
@@ -39,7 +39,7 @@ import java.util.concurrent.ConcurrentHashMap; |
*/ |
@JNINamespace("content") |
public class ChildProcessLauncher { |
- private static final String TAG = "ChildProcessLauncher"; |
+ private static final String TAG = "cr.ChildProcessLaunch"; |
static final int CALLBACK_FOR_UNKNOWN_PROCESS = 0; |
static final int CALLBACK_FOR_GPU_PROCESS = 1; |
@@ -88,8 +88,8 @@ public class ChildProcessLauncher { |
mChildProcessConnections[slot] = new ChildProcessConnectionImpl(context, slot, |
mInSandbox, deathCallback, mChildClass, chromiumLinkerParams, |
alwaysInForeground); |
- Log.d(TAG, "Allocator allocated a connection, sandbox: " + mInSandbox |
- + ", slot: " + slot); |
+ Log.d(TAG, "Allocator allocated a connection, sandbox: %b, slot: %d", mInSandbox, |
+ slot); |
return mChildProcessConnections[slot]; |
} |
} |
@@ -100,15 +100,15 @@ public class ChildProcessLauncher { |
if (mChildProcessConnections[slot] != connection) { |
int occupier = mChildProcessConnections[slot] == null |
? -1 : mChildProcessConnections[slot].getServiceNumber(); |
- Log.e(TAG, "Unable to find connection to free in slot: " + slot |
- + " already occupied by service: " + occupier); |
+ Log.e(TAG, "Unable to find connection to free in slot: %d " |
+ + "already occupied by service: %d", slot, occupier); |
assert false; |
} else { |
mChildProcessConnections[slot] = null; |
assert !mFreeConnectionIndices.contains(slot); |
mFreeConnectionIndices.add(slot); |
- Log.d(TAG, "Allocator freed a connection, sandbox: " + mInSandbox |
- + ", slot: " + slot); |
+ Log.d(TAG, "Allocator freed a connection, sandbox: %b, slot: %d", mInSandbox, |
+ slot); |
} |
} |
} |
@@ -579,8 +579,8 @@ public class ChildProcessLauncher { |
} |
} |
- Log.d(TAG, "Setting up connection to process: slot=" |
- + allocatedConnection.getServiceNumber()); |
+ Log.d(TAG, "Setting up connection to process: slot=%d", |
+ allocatedConnection.getServiceNumber()); |
triggerConnectionSetup(allocatedConnection, commandLine, childProcessId, |
filesToBeMapped, callbackType, clientContext); |
} finally { |
@@ -600,8 +600,8 @@ public class ChildProcessLauncher { |
new ChildProcessConnection.ConnectionCallback() { |
@Override |
public void onConnected(int pid) { |
- Log.d(TAG, "on connect callback, pid=" + pid + " context=" + clientContext |
- + " callbackType=" + callbackType); |
+ Log.d(TAG, "on connect callback, pid=%d context=%d callbackType=%d", |
+ pid, clientContext, callbackType); |
if (pid != NULL_PROCESS_HANDLE) { |
sBindingManager.addNewConnection(pid, connection); |
sServiceMap.put(pid, connection); |
@@ -630,7 +630,7 @@ public class ChildProcessLauncher { |
*/ |
@CalledByNative |
static void stop(int pid) { |
- Log.d(TAG, "stopping child connection: pid=" + pid); |
+ Log.d(TAG, "stopping child connection: pid=%d", pid); |
ChildProcessConnection connection = sServiceMap.remove(pid); |
if (connection == null) { |
logPidWarning(pid, "Tried to stop non-existent connection"); |
@@ -722,7 +722,7 @@ public class ChildProcessLauncher { |
static void logPidWarning(int pid, String message) { |
// This class is effectively a no-op in single process mode, so don't log warnings there. |
if (pid > 0 && !nativeIsSingleProcess()) { |
- Log.w(TAG, message + ", pid=" + pid); |
+ Log.w(TAG, "%s, pid=%d", message, pid); |
} |
} |