Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(696)

Unified Diff: extensions/shell/browser/shell_browser_context.cc

Issue 631203003: Fix bug: AppShell: CHECK failure in PeerConnection init. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..073d419edc4e8326ae2b508d2ff9b8692d0fd424 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());
+ set_url_request_context_getter(
+ new extensions::ShellURLRequestContextGetter(
James Cook 2014/10/07 20:14:17 Don't need "extensions::" here
Xi Han 2014/10/07 21:10:18 Removed.
this,
ignore_certificate_errors_,
GetPath(),
@@ -57,24 +57,24 @@ net::URLRequestContextGetter* ShellBrowserContext::CreateRequestContext(
protocol_handlers,
request_interceptors.Pass(),
net_log_,
- extension_info_map);
- Init();
- return url_request_context_getter_.get();
-}
-
-void ShellBrowserContext::Init(){
- content:: BrowserThread:: PostTask(
+ extension_info_map));
+ resource_context_->set_url_request_context_getter(
+ get_url_request_context_getter());
+ content::BrowserThread::PostTask(
content::BrowserThread::IO,
FROM_HERE,
base::Bind(
- &ShellBrowserContext::InitializationOnIOThread,
+ &ShellBrowserContext::InitURLRequestContextOnIOThread,
base::Unretained(this)));
+ return get_url_request_context_getter();
}
-void ShellBrowserContext::InitializationOnIOThread() {
+void ShellBrowserContext::InitURLRequestContextOnIOThread() {
DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::IO));
- url_request_context_getter_->GetURLRequestContext();
+ // GetURLRequestContext() will create a URLRequestContext if it isn't
+ // initialized.
+ get_url_request_context_getter()->GetURLRequestContext();
}
void ShellBrowserContext::ProfileFunctionCallOnNonProfileBrowserContext1() {

Powered by Google App Engine
This is Rietveld 408576698