Modifying the BackgroundModeManager to handle multiple profiles.

chrome/test/testing_browser_process.cc

Index: chrome/test/testing_browser_process.cc
===================================================================
--- chrome/test/testing_browser_process.cc	(revision 86295)
+++ chrome/test/testing_browser_process.cc	(working copy)
@@ -4,8 +4,10 @@
 
 #include "chrome/test/testing_browser_process.h"
 
+#include "base/command_line.h"
 #include "base/string_util.h"
 #include "base/synchronization/waitable_event.h"
+#include "chrome/browser/background_mode_manager.h"
 #include "chrome/browser/google/google_url_tracker.h"
 #include "chrome/browser/notifications/notification_ui_manager.h"
 #include "chrome/browser/policy/browser_policy_connector.h"
@@ -123,6 +125,18 @@
   return NULL;
 }
 
+BackgroundModeManager* TestingBrowserProcess::background_mode_manager() {
+  if (!background_mode_manager_.get()) {
+    CommandLine cl(CommandLine::NO_PROGRAM);
+    background_mode_manager_.reset(new BackgroundModeManager(&cl, true));

[Andrew T Wilson (Slow), 2011/05/24 20:49:28]

Not sure why we aren't returning NULL here any more? Seems like we really would
rather not create BackgroundModeManager in tests if we can avoid it.

[rpetterson, 2011/05/24 21:48:43]

Done.

+  }
+  return background_mode_manager_.get();
+}
+
+StatusTray* TestingBrowserProcess::status_tray() {
+  return NULL;
+}
+
 safe_browsing::ClientSideDetectionService*
 TestingBrowserProcess::safe_browsing_detection_service() {
   return NULL;