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

Side by Side Diff: content/renderer/raster_worker_pool_unittest.cc

Issue 1616953003: content: Improve thread priority for raster threads. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years, 10 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 "base/test/sequenced_task_runner_test_template.h" 5 #include "base/test/sequenced_task_runner_test_template.h"
6 #include "base/test/task_runner_test_template.h" 6 #include "base/test/task_runner_test_template.h"
7 #include "base/threading/simple_thread.h" 7 #include "base/threading/simple_thread.h"
8 #include "cc/test/task_graph_runner_test_template.h" 8 #include "cc/test/task_graph_runner_test_template.h"
9 #include "content/renderer/raster_worker_pool.h" 9 #include "content/renderer/raster_worker_pool.h"
10 10
11 namespace base { 11 namespace base {
12 namespace { 12 namespace {
13 13
14 // Number of threads spawned in tests. 14 // Number of threads spawned in tests.
15 const int kNumThreads = 4; 15 const int kNumThreads = 4;
16 16
17 class RasterWorkerPoolTestDelegate { 17 class RasterWorkerPoolTestDelegate {
18 public: 18 public:
19 RasterWorkerPoolTestDelegate() 19 RasterWorkerPoolTestDelegate()
20 : raster_worker_pool_(new content::RasterWorkerPool()) {} 20 : raster_worker_pool_(new content::RasterWorkerPool()) {}
21 21
22 void StartTaskRunner() { 22 void StartTaskRunner() {
23 raster_worker_pool_->Start(kNumThreads, SimpleThread::Options()); 23 raster_worker_pool_->Start(kNumThreads);
24 } 24 }
25 25
26 scoped_refptr<content::RasterWorkerPool> GetTaskRunner() { 26 scoped_refptr<content::RasterWorkerPool> GetTaskRunner() {
27 return raster_worker_pool_; 27 return raster_worker_pool_;
28 } 28 }
29 29
30 void StopTaskRunner() { raster_worker_pool_->FlushForTesting(); } 30 void StopTaskRunner() { raster_worker_pool_->FlushForTesting(); }
31 31
32 ~RasterWorkerPoolTestDelegate() { raster_worker_pool_->Shutdown(); } 32 ~RasterWorkerPoolTestDelegate() { raster_worker_pool_->Shutdown(); }
33 33
34 private: 34 private:
35 scoped_refptr<content::RasterWorkerPool> raster_worker_pool_; 35 scoped_refptr<content::RasterWorkerPool> raster_worker_pool_;
36 }; 36 };
37 37
38 INSTANTIATE_TYPED_TEST_CASE_P(RasterWorkerPool, 38 INSTANTIATE_TYPED_TEST_CASE_P(RasterWorkerPool,
39 TaskRunnerTest, 39 TaskRunnerTest,
40 RasterWorkerPoolTestDelegate); 40 RasterWorkerPoolTestDelegate);
41 41
42 class RasterWorkerPoolSequencedTestDelegate { 42 class RasterWorkerPoolSequencedTestDelegate {
43 public: 43 public:
44 RasterWorkerPoolSequencedTestDelegate() 44 RasterWorkerPoolSequencedTestDelegate()
45 : raster_worker_pool_(new content::RasterWorkerPool()) {} 45 : raster_worker_pool_(new content::RasterWorkerPool()) {}
46 46
47 void StartTaskRunner() { 47 void StartTaskRunner() {
48 raster_worker_pool_->Start(kNumThreads, SimpleThread::Options()); 48 raster_worker_pool_->Start(kNumThreads);
49 } 49 }
50 50
51 scoped_refptr<base::SequencedTaskRunner> GetTaskRunner() { 51 scoped_refptr<base::SequencedTaskRunner> GetTaskRunner() {
52 return raster_worker_pool_->CreateSequencedTaskRunner(); 52 return raster_worker_pool_->CreateSequencedTaskRunner();
53 } 53 }
54 54
55 void StopTaskRunner() { raster_worker_pool_->FlushForTesting(); } 55 void StopTaskRunner() { raster_worker_pool_->FlushForTesting(); }
56 56
57 ~RasterWorkerPoolSequencedTestDelegate() { raster_worker_pool_->Shutdown(); } 57 ~RasterWorkerPoolSequencedTestDelegate() { raster_worker_pool_->Shutdown(); }
58 58
(...skipping 11 matching lines...) Expand all
70 namespace cc { 70 namespace cc {
71 namespace { 71 namespace {
72 72
73 template <int NumThreads> 73 template <int NumThreads>
74 class RasterWorkerPoolTaskGraphRunnerTestDelegate { 74 class RasterWorkerPoolTaskGraphRunnerTestDelegate {
75 public: 75 public:
76 RasterWorkerPoolTaskGraphRunnerTestDelegate() 76 RasterWorkerPoolTaskGraphRunnerTestDelegate()
77 : raster_worker_pool_(new content::RasterWorkerPool()) {} 77 : raster_worker_pool_(new content::RasterWorkerPool()) {}
78 78
79 void StartTaskGraphRunner() { 79 void StartTaskGraphRunner() {
80 raster_worker_pool_->Start(NumThreads, base::SimpleThread::Options()); 80 raster_worker_pool_->Start(NumThreads);
81 } 81 }
82 82
83 cc::TaskGraphRunner* GetTaskGraphRunner() { 83 cc::TaskGraphRunner* GetTaskGraphRunner() {
84 return raster_worker_pool_->GetTaskGraphRunner(); 84 return raster_worker_pool_->GetTaskGraphRunner();
85 } 85 }
86 86
87 void StopTaskGraphRunner() { raster_worker_pool_->FlushForTesting(); } 87 void StopTaskGraphRunner() { raster_worker_pool_->FlushForTesting(); }
88 88
89 ~RasterWorkerPoolTaskGraphRunnerTestDelegate() { 89 ~RasterWorkerPoolTaskGraphRunnerTestDelegate() {
90 raster_worker_pool_->Shutdown(); 90 raster_worker_pool_->Shutdown();
(...skipping 20 matching lines...) Expand all
111 TaskGraphRunnerTest, 111 TaskGraphRunnerTest,
112 RasterWorkerPoolTaskGraphRunnerTestDelegate<5>); 112 RasterWorkerPoolTaskGraphRunnerTestDelegate<5>);
113 113
114 // Single threaded tests. 114 // Single threaded tests.
115 INSTANTIATE_TYPED_TEST_CASE_P(RasterWorkerPool, 115 INSTANTIATE_TYPED_TEST_CASE_P(RasterWorkerPool,
116 SingleThreadTaskGraphRunnerTest, 116 SingleThreadTaskGraphRunnerTest,
117 RasterWorkerPoolTaskGraphRunnerTestDelegate<1>); 117 RasterWorkerPoolTaskGraphRunnerTestDelegate<1>);
118 118
119 } // namespace 119 } // namespace
120 } // namespace cc 120 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698