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

Unified Diff: trunk/src/content/browser/shared_worker/shared_worker_service_impl.h

Issue 195723003: Revert 256305 "Add SharedWorkerServiceImpl::CheckWorkerDependenc..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 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: trunk/src/content/browser/shared_worker/shared_worker_service_impl.h
===================================================================
--- trunk/src/content/browser/shared_worker/shared_worker_service_impl.h (revision 256322)
+++ trunk/src/content/browser/shared_worker/shared_worker_service_impl.h (working copy)
@@ -5,8 +5,6 @@
#ifndef CONTENT_BROWSER_SHARED_WORKER_SHARED_WORKER_SERVICE_IMPL_H_
#define CONTENT_BROWSER_SHARED_WORKER_SHARED_WORKER_SERVICE_IMPL_H_
-#include <set>
-
#include "base/compiler_specific.h"
#include "base/containers/scoped_ptr_hash_map.h"
#include "base/memory/singleton.h"
@@ -87,23 +85,12 @@
void OnSharedWorkerMessageFilterClosing(
SharedWorkerMessageFilter* filter);
- // Checks the worker dependency of renderer processes and calls
- // IncrementWorkerRefCount and DecrementWorkerRefCount of
- // RenderProcessHostImpl on UI thread if necessary.
- void CheckWorkerDependency();
-
private:
friend struct DefaultSingletonTraits<SharedWorkerServiceImpl>;
- friend class SharedWorkerServiceImplTest;
- typedef void (*UpdateWorkerDependencyFunc)(const std::vector<int>&,
- const std::vector<int>&);
-
SharedWorkerServiceImpl();
virtual ~SharedWorkerServiceImpl();
- void ResetForTesting();
-
SharedWorkerHost* FindSharedWorkerHost(
SharedWorkerMessageFilter* filter,
int worker_route_id);
@@ -114,16 +101,6 @@
const WorkerStoragePartition& worker_partition,
ResourceContext* resource_context);
- // Returns the IDs of the renderer processes which are executing
- // SharedWorkers connected to other renderer processes.
- const std::set<int> GetRenderersWithWorkerDependency();
-
- void ChangeUpdateWorkerDependencyFuncForTesting(
- UpdateWorkerDependencyFunc new_func);
-
- std::set<int> last_worker_depended_renderers_;
- UpdateWorkerDependencyFunc update_worker_dependency_;
-
// Pair of render_process_id and worker_route_id.
typedef std::pair<int, int> ProcessRouteIdPair;
typedef base::ScopedPtrHashMap<ProcessRouteIdPair,

Powered by Google App Engine
This is Rietveld 408576698