Merge "Am: Clean mLaunchingProviders of all providers started by the client"

Jean-Baptiste Queru · android code review · commit 380a8e3d1263 · 2012-09-05T11:40:30.000-07:00
diff --git a/services/java/com/android/server/am/ActivityManagerService.java b/services/java/com/android/server/am/ActivityManagerService.java
@@ -10961,7 +10961,9 @@ boolean checkAppInLaunchingProvidersLocked(ProcessRecord app, boolean alwaysBad)
                     restart = true;
                 } else {
                     removeDyingProviderLocked(app, cpr, true);
+                    // cpr should have been removed from mLaunchingProviders
                     NL = mLaunchingProviders.size();
+                    i--;
                 }
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -10961,7 +10961,9 @@ boolean checkAppInLaunchingProvidersLocked(ProcessRecord app, boolean alwaysBad)`
`10961`	`10961`	`restart = true;`
`10962`	`10962`	`} else {`
`10963`	`10963`	`removeDyingProviderLocked(app, cpr, true);`
	`10964`	`+ // cpr should have been removed from mLaunchingProviders`
`10964`	`10965`	`NL = mLaunchingProviders.size();`
	`10966`	`+ i--;`
`10965`	`10967`	`}`
`10966`	`10968`	`}`
`10967`	`10969`	`}`