Fix bug: AppShell: CHECK failure in PeerConnection init.

extensions/shell/browser/shell_browser_context.cc

Index: extensions/shell/browser/shell_browser_context.cc
diff --git a/extensions/shell/browser/shell_browser_context.cc b/extensions/shell/browser/shell_browser_context.cc
index 8659de1ec59f8eb88a333e89707a6cc8bf16d2cd..31a412cdda49a20b5e1a762aed5ba9e9ffe2538e 100644
--- a/extensions/shell/browser/shell_browser_context.cc
+++ b/extensions/shell/browser/shell_browser_context.cc
@@ -7,11 +7,11 @@
 #include "base/command_line.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/common/content_switches.h"
-#include "extensions/browser/extension_network_delegate.h"
-#include "extensions/browser/extension_url_request_context_getter.h"
+#include "content/shell/common/shell_switches.h"
 #include "extensions/browser/guest_view/guest_view_manager.h"
-#include "extensions/common/switches.h"
+#include "extensions/shell/browser/shell_network_delegate.h"
 #include "extensions/shell/browser/shell_special_storage_policy.h"
+#include "extensions/shell/browser/shell_url_request_context_getter.h"
 
 namespace extensions {
 
@@ -44,9 +44,9 @@ net::URLRequestContextGetter* ShellBrowserContext::CreateRequestContext(
       content::ProtocolHandlerMap* protocol_handlers,
       content::URLRequestInterceptorScopedVector request_interceptors,
       InfoMap* extension_info_map) {
-  DCHECK(!url_request_context_getter_.get());
-  url_request_context_getter_ =
-      new extensions::ExtensionURLRequestContextGetter(
+  DCHECK(!get_url_request_context_getter());
+  extensions::ShellURLRequestContextGetter* url_request_context_getter =
+      new extensions::ShellURLRequestContextGetter(
           this,
           ignore_certificate_errors_,
           GetPath(),
@@ -58,8 +58,9 @@ net::URLRequestContextGetter* ShellBrowserContext::CreateRequestContext(
           request_interceptors.Pass(),
           net_log_,
           extension_info_map);
+  set_url_request_context_getter(url_request_context_getter);
   Init();
-  return url_request_context_getter_.get();
+  return get_url_request_context_getter();
 }
 
 void ShellBrowserContext::Init(){
@@ -74,7 +75,7 @@ void ShellBrowserContext::Init(){
 void ShellBrowserContext::InitializationOnIOThread() {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::IO));
 
-  url_request_context_getter_->GetURLRequestContext();
+  get_url_request_context_getter()->GetURLRequestContext();

[James Cook, 2014/10/07 16:24:54]

This needs a comment explaining that GetURLRequestContext() has the side-effect
of creating the context.

These initialization functions seem really weird to me.  Init() isn't really
needed.  Also "Init()" implies the whole class is being initialized.  I would:

Eliminate Init().
Rename InitializationOnIOThread() to InitURLRequestContextOnIOThread() or
something similar to make it clear that's the only piece being initialized.

[Xi Han, 2014/10/07 19:29:53]

Remove Init() and rename.

 }
 
 void ShellBrowserContext::ProfileFunctionCallOnNonProfileBrowserContext1() {