Move PromoResourceService from Profile to BrowserProcessImpl/local_state().

chrome/test/base/testing_browser_process.cc

Index: chrome/test/base/testing_browser_process.cc
diff --git a/chrome/test/base/testing_browser_process.cc b/chrome/test/base/testing_browser_process.cc
index 0054fc8adf9dd777a557d22da65fd68c057d4730..c5814373665e787f7b92743a668e1ef32e533edc 100644
--- a/chrome/test/base/testing_browser_process.cc
+++ b/chrome/test/base/testing_browser_process.cc
@@ -6,6 +6,7 @@
 
 #include "base/string_util.h"
 #include "build/build_config.h"
+#include "chrome/browser/browser_process.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/bookmarks/bookmark_prompt_controller.h"
@@ -29,6 +30,11 @@
 #include "chrome/browser/policy/policy_service_stub.h"
 #endif  // defined(ENABLE_CONFIGURATION_POLICY)
 
+// static
+TestingBrowserProcess* TestingBrowserProcess::GetGlobal() {

[Robert Sesek, 2013/01/04 01:16:13]

I'm not sure this is right. Also, who's setting up g_browser_process?

[Dan Beam, 2013/01/04 01:34:42]

I talked with jcivelli@ about this (a chrome/test OWNER) and he was OK with it
(and updating everywhere else).  g_browser_process comes up in various places
(chrome/browser/browser_process_impl.cc, chrome/test/base/chrome_test_suite.cc),
but this really doesn't change anything about how the current code works, it
just makes the calling code prettier.

+  return static_cast<TestingBrowserProcess*>(g_browser_process);
+}
+
 TestingBrowserProcess::TestingBrowserProcess()
     : notification_service_(content::NotificationService::Create()),
       module_ref_count_(0),