Extended: Add "system" URLRequestContext

chrome/browser/browser_process_impl.cc

Index: chrome/browser/browser_process_impl.cc
diff --git a/chrome/browser/browser_process_impl.cc b/chrome/browser/browser_process_impl.cc
index 61dacf389289e8022d2781f786a337e334f9489b..e99c6dbb4f8c13c4e4d4cbb0aecf0ed75872a65f 100644
--- a/chrome/browser/browser_process_impl.cc
+++ b/chrome/browser/browser_process_impl.cc
@@ -54,6 +54,8 @@
 #include "chrome/common/extensions/extension_l10n_util.h"
 #include "chrome/common/extensions/extension_resource.h"
 #include "chrome/common/json_pref_store.h"
+#include "chrome/common/net/url_request_context_getter.h"
+#include "chrome/common/notification_service.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/switch_utils.h"
 #include "chrome/common/url_constants.h"
@@ -77,6 +79,10 @@
 #include "content/common/child_process_messages.h"
 #endif
 
+#if defined(OS_CHROMEOS)
+#include "chrome/browser/chromeos/proxy_config_service_impl.h"
+#endif  // defined(OS_CHROMEOS)
+
 #if (defined(OS_WIN) || defined(OS_LINUX)) && !defined(OS_CHROMEOS)
 // How often to check if the persistent instance of Chrome needs to restart
 // to install an update.
@@ -438,6 +444,22 @@ ui::Clipboard* BrowserProcessImpl::clipboard() {
   return clipboard_.get();
 }
 
+URLRequestContextGetter* BrowserProcessImpl::system_request_context() {
+  DCHECK(CalledOnValidThread());
+  return io_thread()->system_url_request_context_getter();
+}
+
+#if defined(OS_CHROMEOS)
+chromeos::ProxyConfigServiceImpl*
+BrowserProcessImpl::chromeos_proxy_config_service_impl() {

[willchan no longer on Chromium, 2011/03/10 00:40:02]

DCHECK(CalledOnValidThread())

Perhaps you should have a created_foo_ bool like the others to make sure this
isn't double initialized?

[battre, 2011/03/10 16:11:57]

I have added the DCHECK. I think there is no need for a created_foo_ bool:
The chromeos_proxy_config_service_impl_ variable is initialized only once and !=
NULL afterwards. We need the created_foo_ booleans only for variables that are
manually reset in the destructor at times when consumers of BrowserProcessImpl
could still try to access them. In this case the following sequence would be
possible and justifies the created_foo_ booleans:
- destructor is called
  - variable foo_ is reset to zero
- consumer of BrowserProcessImpl calls BrowserBrocessImpl::foo()
--> foo_ is initialized a second time

This cannot happen for BrowserProcessImpl.

I can add the created_foo_ variable but I think there is no benefit.

+  if (!chromeos_proxy_config_service_impl_) {
+    chromeos_proxy_config_service_impl_ =
+        new chromeos::ProxyConfigServiceImpl();
+  }
+  return chromeos_proxy_config_service_impl_;
+}
+#endif  // defined(OS_CHROMEOS)
+
 ExtensionEventRouterForwarder*
 BrowserProcessImpl::extension_event_router_forwarder() {
   return extension_event_router_forwarder_.get();