No longer start BG mode until a BackgroundContents is loaded

chrome/browser/background/background_mode_manager.h

Index: chrome/browser/background/background_mode_manager.h
diff --git a/chrome/browser/background/background_mode_manager.h b/chrome/browser/background/background_mode_manager.h
index 633c92bad4a5cd12bb5fd2044c2ceb4391e63fe2..0402e7921f88908095bf69ec4347c80967edfbc5 100644
--- a/chrome/browser/background/background_mode_manager.h
+++ b/chrome/browser/background/background_mode_manager.h
@@ -45,7 +45,6 @@ class BackgroundModeManager
     : public content::NotificationObserver,
       public BackgroundApplicationListModel::Observer,
       public ProfileInfoCacheObserver,
-      public ProfileKeyedService,
       public ui::SimpleMenuModel::Delegate {
  public:
   BackgroundModeManager(CommandLine* command_line,
@@ -59,12 +58,16 @@ class BackgroundModeManager
   static void LaunchBackgroundApplication(Profile* profile,
                                           const Extension* extension);
 
+  // Returns true if background mode is active. Used only by tests.
+  bool IsBackgroundModeActiveForTest();
+
   // For testing purposes.
   int NumberOfBackgroundModeData();
 
  private:
-  friend class TestBackgroundModeManager;
+  friend class AppBackgroundPageApiTest;

[Mihai Parparita -not on Chrome, 2012/05/03 00:54:11]

Given this friend declaration, does IsBackgroundModeActiveForTest need to be
public?

[Andrew T Wilson (Slow), 2012/05/03 07:42:56]

Agreed, it can be private.

   friend class BackgroundModeManagerTest;
+  friend class TestBackgroundModeManager;
   FRIEND_TEST_ALL_PREFIXES(BackgroundModeManagerTest,
                            BackgroundAppLoadUnload);
   FRIEND_TEST_ALL_PREFIXES(BackgroundModeManagerTest,