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

Unified Diff: cc/test/scheduler_test_common.cc

Issue 1765723002: Hoist begin frame sources out of scheduler (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@scheduler_remove_throttle_flag
Patch Set: Rebase Created 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « cc/test/scheduler_test_common.h ('k') | cc/trees/proxy_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/test/scheduler_test_common.cc
diff --git a/cc/test/scheduler_test_common.cc b/cc/test/scheduler_test_common.cc
index ef760d2adecdf665697d741bca21d4867d27f5c9..1c4ddc7b3b0b1d1f8b598d04b93ed4530008d285 100644
--- a/cc/test/scheduler_test_common.cc
+++ b/cc/test/scheduler_test_common.cc
@@ -181,45 +181,19 @@ base::TimeDelta FakeCompositorTimingHistory::DrawDurationEstimate() const {
return draw_duration_;
}
-scoped_ptr<TestScheduler> TestScheduler::Create(
- base::SimpleTestTickClock* now_src,
- SchedulerClient* client,
- const SchedulerSettings& settings,
- int layer_tree_host_id,
- OrderedSimpleTaskRunner* task_runner,
- BeginFrameSource* external_frame_source,
- scoped_ptr<CompositorTimingHistory> compositor_timing_history) {
- scoped_ptr<TestSyntheticBeginFrameSource> synthetic_frame_source;
- if (!settings.use_external_begin_frame_source) {
- synthetic_frame_source = TestSyntheticBeginFrameSource::Create(
- now_src, task_runner, BeginFrameArgs::DefaultInterval());
- }
- scoped_ptr<TestBackToBackBeginFrameSource> unthrottled_frame_source =
- TestBackToBackBeginFrameSource::Create(now_src, task_runner);
- return make_scoped_ptr(new TestScheduler(
- now_src, client, settings, layer_tree_host_id, task_runner,
- external_frame_source, std::move(synthetic_frame_source),
- std::move(unthrottled_frame_source),
- std::move(compositor_timing_history)));
-}
-
TestScheduler::TestScheduler(
base::SimpleTestTickClock* now_src,
SchedulerClient* client,
const SchedulerSettings& scheduler_settings,
int layer_tree_host_id,
OrderedSimpleTaskRunner* task_runner,
- BeginFrameSource* external_frame_source,
- scoped_ptr<TestSyntheticBeginFrameSource> synthetic_frame_source,
- scoped_ptr<TestBackToBackBeginFrameSource> unthrottled_frame_source,
+ BeginFrameSource* begin_frame_source,
scoped_ptr<CompositorTimingHistory> compositor_timing_history)
: Scheduler(client,
scheduler_settings,
layer_tree_host_id,
task_runner,
- external_frame_source,
- std::move(synthetic_frame_source),
- std::move(unthrottled_frame_source),
+ begin_frame_source,
std::move(compositor_timing_history)),
now_src_(now_src) {}
« no previous file with comments | « cc/test/scheduler_test_common.h ('k') | cc/trees/proxy_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698