From 356843a2d2b37212792a33529818d692d6b61b10 Mon Sep 17 00:00:00 2001 From: Sadrul Habib Chowdhury Date: Wed, 28 Jan 2015 10:46:43 -0500 Subject: [PATCH] cleanup: ThreadLocalState::run_loop_ is never used, so remove it. WorkerTaskRunner manages and uses the WebWorkerRunLoops in a map in |loop_map_|, and |run_loop_| in ThreadLocalState remain unused. So remove it. BUG=none R=avi@chromium.org Review URL: https://codereview.chromium.org/880213002 Cr-Commit-Position: refs/heads/master@{#313519} --- content/child/worker_task_runner.cc | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/content/child/worker_task_runner.cc b/content/child/worker_task_runner.cc index 02c4bab320f4..df4e03132e38 100644 --- a/content/child/worker_task_runner.cc +++ b/content/child/worker_task_runner.cc @@ -30,11 +30,8 @@ class RunClosureTask : public WebWorkerRunLoop::Task { } // namespace struct WorkerTaskRunner::ThreadLocalState { - ThreadLocalState(int id, const WebWorkerRunLoop& loop) - : id_(id), run_loop_(loop) { - } + explicit ThreadLocalState(int id) : id_(id) {} int id_; - WebWorkerRunLoop run_loop_; ObserverList stop_observers_; }; @@ -90,7 +87,7 @@ WorkerTaskRunner::~WorkerTaskRunner() { void WorkerTaskRunner::OnWorkerRunLoopStarted(const WebWorkerRunLoop& loop) { DCHECK(!current_tls_.Get()); int id = id_sequence_.GetNext(); - current_tls_.Set(new ThreadLocalState(id, loop)); + current_tls_.Set(new ThreadLocalState(id)); base::AutoLock locker_(loop_map_lock_); loop_map_[id] = loop; -- 2.11.4.GIT