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

Side by Side Diff: components/scheduler/child/webthread_impl_for_worker_scheduler.h

Issue 1589463002: compositor worker: Use a WebThread for the compositor thread. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix-mandoline-tests Created 4 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 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 #ifndef COMPONENTS_SCHEDULER_CHILD_WEBTHREAD_IMPL_FOR_WORKER_SCHEDULER_H_ 5 #ifndef COMPONENTS_SCHEDULER_CHILD_WEBTHREAD_IMPL_FOR_WORKER_SCHEDULER_H_
6 #define COMPONENTS_SCHEDULER_CHILD_WEBTHREAD_IMPL_FOR_WORKER_SCHEDULER_H_ 6 #define COMPONENTS_SCHEDULER_CHILD_WEBTHREAD_IMPL_FOR_WORKER_SCHEDULER_H_
7 7
8 #include "base/threading/thread.h" 8 #include "base/threading/thread.h"
9 #include "components/scheduler/base/task_queue_manager.h" 9 #include "components/scheduler/base/task_queue_manager.h"
10 #include "components/scheduler/child/webthread_base.h" 10 #include "components/scheduler/child/webthread_base.h"
(...skipping 29 matching lines...) Expand all
40 blink::PlatformThreadId threadId() const override; 40 blink::PlatformThreadId threadId() const override;
41 blink::WebTaskRunner* taskRunner() override; 41 blink::WebTaskRunner* taskRunner() override;
42 42
43 // WebThreadBase implementation. 43 // WebThreadBase implementation.
44 base::SingleThreadTaskRunner* TaskRunner() const override; 44 base::SingleThreadTaskRunner* TaskRunner() const override;
45 scheduler::SingleThreadIdleTaskRunner* IdleTaskRunner() const override; 45 scheduler::SingleThreadIdleTaskRunner* IdleTaskRunner() const override;
46 46
47 // base::MessageLoop::DestructionObserver implementation. 47 // base::MessageLoop::DestructionObserver implementation.
48 void WillDestroyCurrentMessageLoop() override; 48 void WillDestroyCurrentMessageLoop() override;
49 49
50 protected:
51 base::Thread* thread() const { return thread_.get(); }
52
50 private: 53 private:
51 // Returns whether the base::Thread's task-runner should be used as the 54 virtual scoped_ptr<scheduler::WorkerScheduler> CreateWorkerScheduler();
Sami 2016/01/14 13:55:04 Looks like this patch is relative to your older on
sadrul 2016/01/14 15:20:40 Yep, sorry. Done.
52 // default task runner (returns true), or the default task runner from the
53 // scheduler should be used instead (returns false). The default behaviour is
54 // to use the scheduler's default task runner.
55 virtual bool UseThreadTaskRunnerAsDefault() const;
56 55
57 void AddTaskObserverInternal( 56 void AddTaskObserverInternal(
58 base::MessageLoop::TaskObserver* observer) override; 57 base::MessageLoop::TaskObserver* observer) override;
59 void RemoveTaskObserverInternal( 58 void RemoveTaskObserverInternal(
60 base::MessageLoop::TaskObserver* observer) override; 59 base::MessageLoop::TaskObserver* observer) override;
61 60
62 void InitOnThread(base::WaitableEvent* completion); 61 void InitOnThread(base::WaitableEvent* completion);
63 void RestoreTaskRunnerOnThread(base::WaitableEvent* completion); 62 void RestoreTaskRunnerOnThread(base::WaitableEvent* completion);
64 63
65 scoped_ptr<base::Thread> thread_; 64 scoped_ptr<base::Thread> thread_;
66 scoped_ptr<scheduler::WorkerScheduler> worker_scheduler_; 65 scoped_ptr<scheduler::WorkerScheduler> worker_scheduler_;
67 scoped_ptr<scheduler::WebSchedulerImpl> web_scheduler_; 66 scoped_ptr<scheduler::WebSchedulerImpl> web_scheduler_;
68 scoped_refptr<base::SingleThreadTaskRunner> thread_task_runner_; 67 scoped_refptr<base::SingleThreadTaskRunner> thread_task_runner_;
69 scoped_refptr<base::SingleThreadTaskRunner> task_runner_; 68 scoped_refptr<base::SingleThreadTaskRunner> task_runner_;
70 scoped_refptr<scheduler::SingleThreadIdleTaskRunner> idle_task_runner_; 69 scoped_refptr<scheduler::SingleThreadIdleTaskRunner> idle_task_runner_;
71 scoped_refptr<SchedulerTqmDelegate> task_runner_delegate_; 70 scoped_refptr<SchedulerTqmDelegate> task_runner_delegate_;
72 scoped_ptr<WebTaskRunnerImpl> web_task_runner_; 71 scoped_ptr<WebTaskRunnerImpl> web_task_runner_;
73 }; 72 };
74 73
75 } // namespace scheduler 74 } // namespace scheduler
76 75
77 #endif // COMPONENTS_SCHEDULER_CHILD_WEBTHREAD_IMPL_FOR_WORKER_SCHEDULER_H_ 76 #endif // COMPONENTS_SCHEDULER_CHILD_WEBTHREAD_IMPL_FOR_WORKER_SCHEDULER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698