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

Side by Side Diff: cc/surfaces/display_scheduler_unittest.cc

Issue 2632563003: [cc] Calculate the correct latest_confirmed_sequence_number in cc::Scheduler. (Closed)
Patch Set: Fix handling of source_id changes and add some Scheduler tests. Created 3 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 #include "cc/surfaces/display_scheduler.h" 5 #include "cc/surfaces/display_scheduler.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "base/test/null_task_runner.h" 8 #include "base/test/null_task_runner.h"
9 #include "base/test/simple_test_tick_clock.h" 9 #include "base/test/simple_test_tick_clock.h"
10 #include "base/trace_event/trace_event.h" 10 #include "base/trace_event/trace_event.h"
(...skipping 79 matching lines...) Expand 10 before | Expand all | Expand 10 after
90 now_src_.Advance(base::TimeDelta::FromMicroseconds(10000)); 90 now_src_.Advance(base::TimeDelta::FromMicroseconds(10000));
91 scheduler_.SetClient(&client_); 91 scheduler_.SetClient(&client_);
92 } 92 }
93 93
94 ~DisplaySchedulerTest() override {} 94 ~DisplaySchedulerTest() override {}
95 95
96 void SetUp() override { scheduler_.SetRootSurfaceResourcesLocked(false); } 96 void SetUp() override { scheduler_.SetRootSurfaceResourcesLocked(false); }
97 97
98 void AdvanceTimeAndBeginFrameForTest() { 98 void AdvanceTimeAndBeginFrameForTest() {
99 now_src_.Advance(base::TimeDelta::FromMicroseconds(10000)); 99 now_src_.Advance(base::TimeDelta::FromMicroseconds(10000));
100 base::TimeTicks frame_time = now_src_.NowTicks();
101 base::TimeDelta interval = BeginFrameArgs::DefaultInterval();
102 base::TimeTicks deadline = frame_time + interval;
103 // FakeBeginFrameSource deals with |source_id| and |sequence_number|. 100 // FakeBeginFrameSource deals with |source_id| and |sequence_number|.
104 fake_begin_frame_source_.TestOnBeginFrame( 101 BeginFrameArgs args = fake_begin_frame_source_.CreateBeginFrameArgs(
105 BeginFrameArgs::Create(BEGINFRAME_FROM_HERE, 0, 1, frame_time, deadline, 102 BEGINFRAME_FROM_HERE, &now_src_);
106 interval, BeginFrameArgs::NORMAL)); 103 fake_begin_frame_source_.TestOnBeginFrame(args);
107 } 104 }
108 105
109 protected: 106 protected:
110 base::SimpleTestTickClock& now_src() { return now_src_; } 107 base::SimpleTestTickClock& now_src() { return now_src_; }
111 FakeDisplaySchedulerClient& client() { return client_; } 108 FakeDisplaySchedulerClient& client() { return client_; }
112 DisplayScheduler& scheduler() { return scheduler_; } 109 DisplayScheduler& scheduler() { return scheduler_; }
113 110
114 FakeExternalBeginFrameSource fake_begin_frame_source_; 111 FakeExternalBeginFrameSource fake_begin_frame_source_;
115 112
116 base::SimpleTestTickClock now_src_; 113 base::SimpleTestTickClock now_src_;
(...skipping 464 matching lines...) Expand 10 before | Expand all | Expand 10 after
581 578
582 scheduler_.SetRootSurfaceResourcesLocked(true); 579 scheduler_.SetRootSurfaceResourcesLocked(true);
583 EXPECT_EQ(++count, scheduler_.scheduler_begin_frame_deadline_count()); 580 EXPECT_EQ(++count, scheduler_.scheduler_begin_frame_deadline_count());
584 581
585 scheduler_.OutputSurfaceLost(); 582 scheduler_.OutputSurfaceLost();
586 EXPECT_EQ(++count, scheduler_.scheduler_begin_frame_deadline_count()); 583 EXPECT_EQ(++count, scheduler_.scheduler_begin_frame_deadline_count());
587 } 584 }
588 585
589 } // namespace 586 } // namespace
590 } // namespace cc 587 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698