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

Side by Side Diff: content/renderer/scheduler/webthread_impl_for_scheduler_unittest.cc

Issue 1025323003: Introduce a SchedulerHelper in content/child/scheduler (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Sami's comments Created 5 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 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/renderer/scheduler/webthread_impl_for_scheduler.h" 5 #include "content/renderer/scheduler/webthread_impl_for_scheduler.h"
6 6
7 #include "base/run_loop.h" 7 #include "base/run_loop.h"
8 #include "content/child/scheduler/scheduler_message_loop_delegate.h"
8 #include "content/renderer/scheduler/renderer_scheduler_impl.h" 9 #include "content/renderer/scheduler/renderer_scheduler_impl.h"
9 #include "content/renderer/scheduler/renderer_scheduler_message_loop_delegate.h"
10 #include "testing/gmock/include/gmock/gmock.h" 10 #include "testing/gmock/include/gmock/gmock.h"
11 #include "testing/gtest/include/gtest/gtest.h" 11 #include "testing/gtest/include/gtest/gtest.h"
12 #include "third_party/WebKit/public/platform/WebTraceLocation.h" 12 #include "third_party/WebKit/public/platform/WebTraceLocation.h"
13 13
14 namespace content { 14 namespace content {
15 namespace { 15 namespace {
16 16
17 const int kWorkBatchSize = 2; 17 const int kWorkBatchSize = 2;
18 18
19 class MockTask : public blink::WebThread::Task { 19 class MockTask : public blink::WebThread::Task {
20 public: 20 public:
21 MOCK_METHOD0(run, void()); 21 MOCK_METHOD0(run, void());
22 }; 22 };
23 23
24 class MockTaskObserver : public blink::WebThread::TaskObserver { 24 class MockTaskObserver : public blink::WebThread::TaskObserver {
25 public: 25 public:
26 MOCK_METHOD0(willProcessTask, void()); 26 MOCK_METHOD0(willProcessTask, void());
27 MOCK_METHOD0(didProcessTask, void()); 27 MOCK_METHOD0(didProcessTask, void());
28 }; 28 };
29 } // namespace 29 } // namespace
30 30
31 class WebThreadImplForSchedulerTest : public testing::Test { 31 class WebThreadImplForSchedulerTest : public testing::Test {
32 public: 32 public:
33 WebThreadImplForSchedulerTest() 33 WebThreadImplForSchedulerTest()
34 : scheduler_( 34 : scheduler_(SchedulerMessageLoopDelegate::Create(&message_loop_)),
35 RendererSchedulerMessageLoopDelegate::Create(&message_loop_)),
36 default_task_runner_(scheduler_.DefaultTaskRunner()), 35 default_task_runner_(scheduler_.DefaultTaskRunner()),
37 thread_(&scheduler_) {} 36 thread_(&scheduler_) {}
38 37
39 ~WebThreadImplForSchedulerTest() override {} 38 ~WebThreadImplForSchedulerTest() override {}
40 39
40 void SetWorkBatchSizeForTesting(size_t work_batch_size) {
41 scheduler_.SetWorkBatchSizeForTesting(work_batch_size);
42 }
43
41 protected: 44 protected:
42 void EatDefaultTask(MockTaskObserver* observer) { 45 void EatDefaultTask(MockTaskObserver* observer) {
43 // The scheduler posts one extra DoWork() task automatically. 46 // The scheduler posts one extra DoWork() task automatically.
44 EXPECT_CALL(*observer, willProcessTask()); 47 EXPECT_CALL(*observer, willProcessTask());
45 EXPECT_CALL(*observer, didProcessTask()); 48 EXPECT_CALL(*observer, didProcessTask());
46 } 49 }
47 50
48 base::MessageLoop message_loop_; 51 base::MessageLoop message_loop_;
49 RendererSchedulerImpl scheduler_; 52 RendererSchedulerImpl scheduler_;
50 scoped_refptr<base::SingleThreadTaskRunner> default_task_runner_; 53 scoped_refptr<base::SingleThreadTaskRunner> default_task_runner_;
(...skipping 139 matching lines...) Expand 10 before | Expand all | Expand 10 after
190 } 193 }
191 194
192 message_loop_.PostTask( 195 message_loop_.PostTask(
193 FROM_HERE, base::Bind(&EnterRunLoop, base::Unretained(&message_loop_), 196 FROM_HERE, base::Bind(&EnterRunLoop, base::Unretained(&message_loop_),
194 base::Unretained(&thread_))); 197 base::Unretained(&thread_)));
195 message_loop_.RunUntilIdle(); 198 message_loop_.RunUntilIdle();
196 thread_.removeTaskObserver(&observer); 199 thread_.removeTaskObserver(&observer);
197 } 200 }
198 201
199 } // namespace content 202 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698