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

Unified Diff: chrome/browser/renderer_host/resource_message_filter.cc

Issue 509016: Refactored code to allow associating workers with multiple renderers. (Closed)
Patch Set: Disabled overly-aggressive assertion in ResourceDispatcherHost. Created 10 years, 11 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: chrome/browser/renderer_host/resource_message_filter.cc
diff --git a/chrome/browser/renderer_host/resource_message_filter.cc b/chrome/browser/renderer_host/resource_message_filter.cc
index 12959dcb291385c39d2a444d49ef7bdaa5baa4e0..64a40230efcf255638153a158bb287cd5c913f32 100644
--- a/chrome/browser/renderer_host/resource_message_filter.cc
+++ b/chrome/browser/renderer_host/resource_message_filter.cc
@@ -634,26 +634,24 @@ void ResourceMessageFilter::OnLaunchNaCl(
host->Launch(this, channel_descriptor, reply_msg);
}
-void ResourceMessageFilter::OnCreateWorker(const GURL& url,
- bool is_shared,
- const string16& name,
- int render_view_route_id,
- int* route_id) {
+void ResourceMessageFilter::OnCreateWorker(
+ const ViewHostMsg_CreateWorker_Params& params, int* route_id) {
*route_id = render_widget_helper_->GetNextRoutingID();
WorkerService::GetInstance()->CreateWorker(
- url, is_shared, off_the_record(), name, id(), render_view_route_id, this,
- *route_id);
+ params.url, params.is_shared, off_the_record(), params.name,
+ params.document_id, id(), params.render_view_route_id, this, *route_id);
}
void ResourceMessageFilter::OnLookupSharedWorker(const GURL& url,
const string16& name,
unsigned long long document_id,
+ int render_view_route_id,
int* route_id,
bool* url_mismatch) {
int new_route_id = render_widget_helper_->GetNextRoutingID();
bool worker_found = WorkerService::GetInstance()->LookupSharedWorker(
- url, name, off_the_record(), document_id, this, new_route_id,
- url_mismatch);
+ url, name, off_the_record(), document_id, id(), render_view_route_id,
+ this, new_route_id, url_mismatch);
*route_id = worker_found ? new_route_id : MSG_ROUTING_NONE;
}
« no previous file with comments | « chrome/browser/renderer_host/resource_message_filter.h ('k') | chrome/browser/renderer_host/resource_request_details.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698