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

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

Issue 2655463012: Really de-flake ChildProcessLauncherTest#testBindServiceFromMultipleProcesses. (Closed)
Patch Set: Simpler poll criteria. Created 3 years, 11 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/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 3ae8def576b792254f5e7a6df9383ab1dbdbbd96..9b7d6b2ff8179385562062adf06a43ab27be289f 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
@@ -133,6 +133,11 @@ public class ChildProcessLauncher {
int allocatedConnectionsCountForTesting() {
return mChildProcessConnections.length - mFreeConnectionIndices.size();
}
+
+ @VisibleForTesting
+ ChildProcessConnection[] connectionArrayForTesting() {
+ return mChildProcessConnections;
+ }
}
private static class PendingSpawnData {
@@ -946,11 +951,12 @@ public class ChildProcessLauncher {
}
/**
- * @return the service map of connected services
+ * @return gets the service connection array for a specific package name.
*/
@VisibleForTesting
- static Map<Integer, ChildProcessConnection> getServiceMapForTesting() {
- return sServiceMap;
+ static ChildProcessConnection[] getSandboxedConnectionArrayForTesting(
+ String packageName) {
+ return sSandboxedChildConnectionAllocatorMap.get(packageName).connectionArrayForTesting();
}
/** @return the count of services set up and working */

Powered by Google App Engine
This is Rietveld 408576698