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

Side by Side Diff: content/child/scheduler/webthread_impl_for_worker_scheduler.cc

Issue 1088053003: Patch 1/3 to get WebScheduler via WebThread (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added a BaseWebScheduler Created 5 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/child/scheduler/webthread_impl_for_worker_scheduler.h" 5 #include "content/child/scheduler/webthread_impl_for_worker_scheduler.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/synchronization/waitable_event.h" 8 #include "base/synchronization/waitable_event.h"
9 #include "content/child/scheduler/scheduler_message_loop_delegate.h" 9 #include "content/child/scheduler/scheduler_message_loop_delegate.h"
10 #include "content/child/scheduler/worker_scheduler_impl.h" 10 #include "content/child/scheduler/worker_scheduler_impl.h"
11 #include "content/child/scheduler/worker_web_scheduler_impl.h"
11 #include "third_party/WebKit/public/platform/WebTraceLocation.h" 12 #include "third_party/WebKit/public/platform/WebTraceLocation.h"
12 13
13 namespace content { 14 namespace content {
14 15
15 WebThreadImplForWorkerScheduler::WebThreadImplForWorkerScheduler( 16 WebThreadImplForWorkerScheduler::WebThreadImplForWorkerScheduler(
16 const char* name) 17 const char* name)
17 : thread_(new base::Thread(name)) { 18 : thread_(new base::Thread(name)) {
18 thread_->Start(); 19 thread_->Start();
19 20
20 base::WaitableEvent completion(false, false); 21 base::WaitableEvent completion(false, false);
(...skipping 14 matching lines...) Expand all
35 36
36 thread_->Stop(); 37 thread_->Stop();
37 } 38 }
38 39
39 void WebThreadImplForWorkerScheduler::InitOnThread( 40 void WebThreadImplForWorkerScheduler::InitOnThread(
40 base::WaitableEvent* completion) { 41 base::WaitableEvent* completion) {
41 worker_scheduler_ = WorkerScheduler::Create(thread_->message_loop()); 42 worker_scheduler_ = WorkerScheduler::Create(thread_->message_loop());
42 worker_scheduler_->Init(); 43 worker_scheduler_->Init();
43 task_runner_ = worker_scheduler_->DefaultTaskRunner(); 44 task_runner_ = worker_scheduler_->DefaultTaskRunner();
44 idle_task_runner_ = worker_scheduler_->IdleTaskRunner(); 45 idle_task_runner_ = worker_scheduler_->IdleTaskRunner();
46 web_scheduler_.reset(new WorkerWebSchedulerImpl(worker_scheduler_.get()));
45 completion->Signal(); 47 completion->Signal();
46 } 48 }
47 49
48 void WebThreadImplForWorkerScheduler::ShutDownOnThread( 50 void WebThreadImplForWorkerScheduler::ShutDownOnThread(
49 base::WaitableEvent* completion) { 51 base::WaitableEvent* completion) {
50 task_runner_ = nullptr; 52 task_runner_ = nullptr;
51 idle_task_runner_ = nullptr; 53 idle_task_runner_ = nullptr;
54 web_scheduler_.reset(nullptr);
Sami 2015/04/16 10:44:01 nit: I think you can just call reset() without any
alex clarke (OOO till 29th) 2015/04/16 13:35:01 Done.
52 worker_scheduler_.reset(nullptr); 55 worker_scheduler_.reset(nullptr);
53 completion->Signal(); 56 completion->Signal();
54 } 57 }
55 58
56 blink::PlatformThreadId WebThreadImplForWorkerScheduler::threadId() const { 59 blink::PlatformThreadId WebThreadImplForWorkerScheduler::threadId() const {
57 return thread_->thread_id(); 60 return thread_->thread_id();
58 } 61 }
59 62
63 blink::WebScheduler* WebThreadImplForWorkerScheduler::scheduler() const {
64 return web_scheduler_.get();
65 }
66
60 base::MessageLoop* WebThreadImplForWorkerScheduler::MessageLoop() const { 67 base::MessageLoop* WebThreadImplForWorkerScheduler::MessageLoop() const {
61 // As per WebThreadImpl::MessageLoop() 68 // As per WebThreadImpl::MessageLoop()
62 return nullptr; 69 return nullptr;
63 } 70 }
64 71
65 base::SingleThreadTaskRunner* WebThreadImplForWorkerScheduler::TaskRunner() 72 base::SingleThreadTaskRunner* WebThreadImplForWorkerScheduler::TaskRunner()
66 const { 73 const {
67 return task_runner_.get(); 74 return task_runner_.get();
68 } 75 }
69 76
70 SingleThreadIdleTaskRunner* WebThreadImplForWorkerScheduler::IdleTaskRunner() 77 SingleThreadIdleTaskRunner* WebThreadImplForWorkerScheduler::IdleTaskRunner()
71 const { 78 const {
72 return idle_task_runner_.get(); 79 return idle_task_runner_.get();
73 } 80 }
74 81
75 void WebThreadImplForWorkerScheduler::AddTaskObserverInternal( 82 void WebThreadImplForWorkerScheduler::AddTaskObserverInternal(
76 base::MessageLoop::TaskObserver* observer) { 83 base::MessageLoop::TaskObserver* observer) {
77 worker_scheduler_->AddTaskObserver(observer); 84 worker_scheduler_->AddTaskObserver(observer);
78 } 85 }
79 86
80 void WebThreadImplForWorkerScheduler::RemoveTaskObserverInternal( 87 void WebThreadImplForWorkerScheduler::RemoveTaskObserverInternal(
81 base::MessageLoop::TaskObserver* observer) { 88 base::MessageLoop::TaskObserver* observer) {
82 worker_scheduler_->RemoveTaskObserver(observer); 89 worker_scheduler_->RemoveTaskObserver(observer);
83 } 90 }
84 91
85 } // namespace content 92 } // namespace content
OLDNEW
« no previous file with comments | « content/child/scheduler/webthread_impl_for_worker_scheduler.h ('k') | content/child/scheduler/worker_web_scheduler_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698