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

Side by Side Diff: cc/test/scheduler_test_common.h

Issue 1136123009: Remove "mixin" from BeginFrameSource/Observer base classes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: updated comments Created 5 years, 6 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
« no previous file with comments | « cc/test/fake_external_begin_frame_source.h ('k') | cc/test/scheduler_test_common.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2011 The Chromium Authors. All rights reserved. 1 // Copyright 2011 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 CC_TEST_SCHEDULER_TEST_COMMON_H_ 5 #ifndef CC_TEST_SCHEDULER_TEST_COMMON_H_
6 #define CC_TEST_SCHEDULER_TEST_COMMON_H_ 6 #define CC_TEST_SCHEDULER_TEST_COMMON_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 OrderedSimpleTaskRunner* task_runner); 66 OrderedSimpleTaskRunner* task_runner);
67 67
68 // Overridden from DelayBasedTimeSource 68 // Overridden from DelayBasedTimeSource
69 ~TestDelayBasedTimeSource() override; 69 ~TestDelayBasedTimeSource() override;
70 base::TimeTicks Now() const override; 70 base::TimeTicks Now() const override;
71 std::string TypeString() const override; 71 std::string TypeString() const override;
72 72
73 scoped_refptr<TestNowSource> now_src_; 73 scoped_refptr<TestNowSource> now_src_;
74 }; 74 };
75 75
76 struct FakeBeginFrameSource : public BeginFrameSourceMixIn { 76 struct FakeBeginFrameSource : public BeginFrameSourceBase {
77 bool remaining_frames_ = false; 77 bool remaining_frames_ = false;
78 78
79 BeginFrameObserver* GetObserver() { return observer_; } 79 BeginFrameObserver* GetObserver() { return observer_; }
80 80
81 BeginFrameArgs TestLastUsedBeginFrameArgs() { 81 BeginFrameArgs TestLastUsedBeginFrameArgs() {
82 if (observer_) { 82 if (observer_) {
83 return observer_->LastUsedBeginFrameArgs(); 83 return observer_->LastUsedBeginFrameArgs();
84 } 84 }
85 return BeginFrameArgs(); 85 return BeginFrameArgs();
86 } 86 }
(...skipping 124 matching lines...) Expand 10 before | Expand all | Expand 10 after
211 const scoped_refptr<OrderedSimpleTaskRunner>& test_task_runner, 211 const scoped_refptr<OrderedSimpleTaskRunner>& test_task_runner,
212 TestSchedulerFrameSourcesConstructor* frame_sources_constructor, 212 TestSchedulerFrameSourcesConstructor* frame_sources_constructor,
213 scoped_ptr<BeginFrameSource> external_begin_frame_source); 213 scoped_ptr<BeginFrameSource> external_begin_frame_source);
214 214
215 scoped_refptr<TestNowSource> now_src_; 215 scoped_refptr<TestNowSource> now_src_;
216 }; 216 };
217 217
218 } // namespace cc 218 } // namespace cc
219 219
220 #endif // CC_TEST_SCHEDULER_TEST_COMMON_H_ 220 #endif // CC_TEST_SCHEDULER_TEST_COMMON_H_
OLDNEW
« no previous file with comments | « cc/test/fake_external_begin_frame_source.h ('k') | cc/test/scheduler_test_common.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698