Extended: Add "system" URLRequestContext

chrome/browser/browser_process.h

Index: chrome/browser/browser_process.h
diff --git a/chrome/browser/browser_process.h b/chrome/browser/browser_process.h
index d406eb0d479306955e7650fc5fbbc68db20c18e5..6ed719ed172ddf1026fcc64b91f79a6ef7d553ac 100644
--- a/chrome/browser/browser_process.h
+++ b/chrome/browser/browser_process.h
@@ -15,6 +15,7 @@
 #include <vector>
 
 #include "base/basictypes.h"
+#include "base/ref_counted.h"
 #include "ipc/ipc_message.h"
 
 class AutomationProviderList;
@@ -40,6 +41,7 @@ class ResourceDispatcherHost;
 class SidebarManager;
 class TabCloseableStateWatcher;
 class ThumbnailGenerator;
+class URLRequestContextGetter;
 class WatchDogThread;
 
 namespace base {
@@ -47,6 +49,12 @@ class Thread;
 class WaitableEvent;
 }
 
+#if defined(OS_CHROMEOS)
+namespace chromeos {
+class ProxyConfigServiceImpl;
+}
+#endif  // defined(OS_CHROMEOS)
+
 namespace printing {
 class PrintJobManager;
 class PrintPreviewTabController;
@@ -82,6 +90,14 @@ class BrowserProcess {
   virtual DevToolsManager* devtools_manager() = 0;
   virtual SidebarManager* sidebar_manager() = 0;
   virtual ui::Clipboard* clipboard() = 0;
+  virtual scoped_refptr<URLRequestContextGetter> system_request_context() = 0;

[Mattias Nissler (ping if slow), 2011/03/09 11:04:56]

Why do we return this wrapped in a scoped_refptr? Can't the caller just take
care of that if required?

[battre, 2011/03/09 19:24:07]

I think I have copied the pattern from somewhere, but I agree that in most cases
we don't return wrapped pointers. So I have deleted the wrapping.

Done.

+
+#if defined(OS_CHROMEOS)
+  // Returns ChromeOS's ProxyConfigServiceImpl, creating if not yet created.
+  virtual chromeos::ProxyConfigServiceImpl*
+      chromeos_proxy_config_service_impl() = 0;
+#endif  // defined(OS_CHROMEOS)
+
   virtual ExtensionEventRouterForwarder*
       extension_event_router_forwarder() = 0;