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

Unified Diff: content/browser/worker_host/worker_service_impl.cc

Issue 9369009: Make content::ResourceContext be a real interface like the rest of the Content API (i.e. don't ha... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: sync Created 8 years, 10 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
« no previous file with comments | « content/browser/worker_host/worker_service_impl.h ('k') | content/public/browser/browser_context.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/worker_host/worker_service_impl.cc
===================================================================
--- content/browser/worker_host/worker_service_impl.cc (revision 121250)
+++ content/browser/worker_host/worker_service_impl.cc (working copy)
@@ -85,7 +85,7 @@
const ViewHostMsg_CreateWorker_Params& params,
int route_id,
WorkerMessageFilter* filter,
- const ResourceContext& resource_context) {
+ ResourceContext* resource_context) {
// Generate a unique route id for the browser-worker communication that's
// unique among all worker processes. That way when the worker process sends
// a wrapped IPC message through us, we know which WorkerProcessHost to give
@@ -96,7 +96,7 @@
next_worker_route_id(),
0,
params.script_resource_appcache_id,
- &resource_context);
+ resource_context);
instance.AddFilter(filter, route_id);
instance.worker_document_set()->Add(
filter, params.document_id, filter->render_process_id(),
@@ -109,7 +109,7 @@
const ViewHostMsg_CreateWorker_Params& params,
int route_id,
WorkerMessageFilter* filter,
- const ResourceContext* resource_context,
+ ResourceContext* resource_context,
bool* exists,
bool* url_mismatch) {
*exists = true;
@@ -292,7 +292,7 @@
// TODO(michaeln): As written, test can fail per my earlier comment in
// this method, but that's a bug.
- // DCHECK(worker->request_context() == instance.request_context());
+ // DCHECK(worker->request_context() == instance.GetRequestContext());
worker->CreateWorker(instance);
FOR_EACH_OBSERVER(
@@ -506,7 +506,7 @@
WorkerProcessHost::WorkerInstance* WorkerServiceImpl::FindSharedWorkerInstance(
const GURL& url,
const string16& name,
- const ResourceContext* resource_context) {
+ ResourceContext* resource_context) {
for (WorkerProcessHostIterator iter; !iter.Done(); ++iter) {
for (WorkerProcessHost::Instances::iterator instance_iter =
iter->mutable_instances().begin();
@@ -522,7 +522,7 @@
WorkerProcessHost::WorkerInstance* WorkerServiceImpl::FindPendingInstance(
const GURL& url,
const string16& name,
- const ResourceContext* resource_context) {
+ ResourceContext* resource_context) {
// Walk the pending instances looking for a matching pending worker.
for (WorkerProcessHost::Instances::iterator iter =
pending_shared_workers_.begin();
@@ -539,7 +539,7 @@
void WorkerServiceImpl::RemovePendingInstances(
const GURL& url,
const string16& name,
- const ResourceContext* resource_context) {
+ ResourceContext* resource_context) {
// Walk the pending instances looking for a matching pending worker.
for (WorkerProcessHost::Instances::iterator iter =
pending_shared_workers_.begin();
@@ -555,7 +555,7 @@
WorkerProcessHost::WorkerInstance* WorkerServiceImpl::CreatePendingInstance(
const GURL& url,
const string16& name,
- const ResourceContext* resource_context) {
+ ResourceContext* resource_context) {
// Look for an existing pending shared worker.
WorkerProcessHost::WorkerInstance* instance =
FindPendingInstance(url, name, resource_context);
« no previous file with comments | « content/browser/worker_host/worker_service_impl.h ('k') | content/public/browser/browser_context.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698