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

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

Issue 133093003: Move the worker script loading code to the worker process (phase:4/5) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@02ScriptLoadInWorkerChrome
Patch Set: change comments in worker_webapplicationcachehost_impl.h and rebase. Created 6 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: content/browser/worker_host/worker_service_impl.h
diff --git a/content/browser/worker_host/worker_service_impl.h b/content/browser/worker_host/worker_service_impl.h
index f7546eb0fdac3093434c0fe023c8e6b146a9ac00..133c0a38e1a8a982bb675521c4722b7884fb15a8 100644
--- a/content/browser/worker_host/worker_service_impl.h
+++ b/content/browser/worker_host/worker_service_impl.h
@@ -44,14 +44,8 @@ class CONTENT_EXPORT WorkerServiceImpl
int route_id,
WorkerMessageFilter* filter,
ResourceContext* resource_context,
- const WorkerStoragePartition& worker_partition);
- void LookupSharedWorker(const ViewHostMsg_CreateWorker_Params& params,
- int route_id,
- WorkerMessageFilter* filter,
- ResourceContext* resource_context,
- const WorkerStoragePartition& worker_partition,
- bool* exists,
- bool* url_error);
+ const WorkerStoragePartition& worker_partition,
+ bool* url_mismatch);
void ForwardToWorker(const IPC::Message& message,
WorkerMessageFilter* filter);
void DocumentDetached(unsigned long long document_id,
@@ -105,23 +99,6 @@ class CONTENT_EXPORT WorkerServiceImpl
// Tries to see if any of the queued workers can be created.
void TryStartingQueuedWorker();
- // APIs for manipulating our set of pending shared worker instances.
- WorkerProcessHost::WorkerInstance* CreatePendingInstance(
- const GURL& url,
- const base::string16& name,
- ResourceContext* resource_context,
- const WorkerStoragePartition& worker_partition);
- WorkerProcessHost::WorkerInstance* FindPendingInstance(
- const GURL& url,
- const base::string16& name,
- const WorkerStoragePartition& worker_partition,
- ResourceContext* resource_context);
- void RemovePendingInstances(
- const GURL& url,
- const base::string16& name,
- const WorkerStoragePartition& worker_partition,
- ResourceContext* resource_context);
-
WorkerProcessHost::WorkerInstance* FindSharedWorkerInstance(
const GURL& url,
const base::string16& name,
@@ -134,11 +111,6 @@ class CONTENT_EXPORT WorkerServiceImpl
WorkerProcessHost::Instances queued_workers_;
- // These are shared workers that have been looked up, but not created yet.
- // We need to keep a list of these to synchronously detect shared worker
- // URL mismatches when two pages launch shared workers simultaneously.
- WorkerProcessHost::Instances pending_shared_workers_;
-
ObserverList<WorkerServiceObserver> observers_;
DISALLOW_COPY_AND_ASSIGN(WorkerServiceImpl);

Powered by Google App Engine
This is Rietveld 408576698