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

Unified Diff: content/browser/renderer_host/resource_dispatcher_host.cc

Issue 6576020: Remove Gears from Chrome (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: windows fixes Created 9 years, 9 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: content/browser/renderer_host/resource_dispatcher_host.cc
diff --git a/content/browser/renderer_host/resource_dispatcher_host.cc b/content/browser/renderer_host/resource_dispatcher_host.cc
index aebdbb045e7a7e7503d95e542fcce1b5383f1501..bd9f1ec1336827b1b9008f886c11f9bba1f7bfd8 100644
--- a/content/browser/renderer_host/resource_dispatcher_host.cc
+++ b/content/browser/renderer_host/resource_dispatcher_host.cc
@@ -360,7 +360,7 @@ void ResourceDispatcherHost::BeginRequest(
int child_id = filter_->child_id();
ChromeURLRequestContext* context = filter_->GetURLRequestContext(
- request_data);
+ request_data.resource_type);
// Might need to resolve the blob references in the upload data.
if (request_data.upload_data && context) {
@@ -389,12 +389,6 @@ void ResourceDispatcherHost::BeginRequest(
return;
}
- // Ensure the Chrome plugins are loaded, as they may intercept network
- // requests. Does nothing if they are already loaded.
- // TODO(mpcomplete): This takes 200 ms! Investigate parallelizing this by
- // starting the load earlier in a BG thread.
- PluginService::GetInstance()->LoadChromePlugins(this);
-
// Construct the event handler.
scoped_refptr<ResourceHandler> handler;
if (sync_result) {
@@ -518,9 +512,7 @@ void ResourceDispatcherHost::BeginRequest(
upload_size,
false, // is download
ResourceType::IsFrame(request_data.resource_type), // allow_download
- request_data.has_user_gesture,
- request_data.host_renderer_id,
- request_data.host_render_view_id);
+ request_data.has_user_gesture);
SetRequestInfo(request, extra_info); // Request takes ownership.
chrome_browser_net::SetOriginPIDForRequest(
request_data.origin_pid, request);
@@ -647,9 +639,7 @@ ResourceDispatcherHost::CreateRequestInfoForBrowserRequest(
0, // upload_size
download, // is_download
download, // allow_download
- false, // has_user_gesture
- -1, // host renderer id
- -1); // host render view id
+ false); // has_user_gesture
}
void ResourceDispatcherHost::OnClosePageACK(
@@ -701,9 +691,6 @@ void ResourceDispatcherHost::BeginDownload(
NewRunnableFunction(&download_util::NotifyDownloadInitiated,
child_id, route_id));
- // Ensure the Chrome plugins are loaded, as they may intercept network
- // requests. Does nothing if they are already loaded.
- PluginService::GetInstance()->LoadChromePlugins(this);
net::URLRequest* request = new net::URLRequest(url, this);
request_id_--;
@@ -754,10 +741,6 @@ void ResourceDispatcherHost::BeginSaveFile(
if (is_shutdown_)
return;
- // Ensure the Chrome plugins are loaded, as they may intercept network
- // requests. Does nothing if they are already loaded.
- PluginService::GetInstance()->LoadChromePlugins(this);
-
scoped_refptr<ResourceHandler> handler(
new SaveFileResourceHandler(child_id,
route_id,

Powered by Google App Engine
This is Rietveld 408576698