Now closes existing background contents if app opens a new one.

chrome/browser/background/background_contents_service.cc

Index: chrome/browser/background/background_contents_service.cc
diff --git a/chrome/browser/background/background_contents_service.cc b/chrome/browser/background/background_contents_service.cc
index bc3e6062d18e3acdb8dd3919983c16d67f048ab2..748f272350002b79b2f46e0768f6c56e3336cdfe 100644
--- a/chrome/browser/background/background_contents_service.cc
+++ b/chrome/browser/background/background_contents_service.cc
@@ -357,7 +357,9 @@ void BackgroundContentsService::LoadBackgroundContentsFromPrefs(
       // crash before we could save our prefs.
       NOTREACHED() << "No extension found for BackgroundContents - id = "
                    << *it;
-      return;
+      // Don't cancel out of our loop, just ignore this BackgroundContents and
+      // load the next one.
+      continue;

[rafaelw, 2011/10/04 20:10:28]

It seems wrong to leave the NOTREACHED() here. Also, if reaching this point is
expectable, the comment should explain how it happens.

[Andrew T Wilson (Slow), 2011/10/04 21:12:05]

Updated the comment. Let me know if you want this changed to a DLOG(ERROR).

     }
     LoadBackgroundContentsFromDictionary(profile, *it, contents);
   }