| Index: cc/scheduler/compositor_timing_history_unittest.cc
|
| diff --git a/cc/scheduler/compositor_timing_history_unittest.cc b/cc/scheduler/compositor_timing_history_unittest.cc
|
| index 900367f14ec5152c2502b73118da017b7328cf66..f57d44340c5a3a51b83472e2378afdbaea6d9a76 100644
|
| --- a/cc/scheduler/compositor_timing_history_unittest.cc
|
| +++ b/cc/scheduler/compositor_timing_history_unittest.cc
|
| @@ -17,7 +17,8 @@ class TestCompositorTimingHistory : public CompositorTimingHistory {
|
| public:
|
| TestCompositorTimingHistory(CompositorTimingHistoryTest* test,
|
| RenderingStatsInstrumentation* rendering_stats)
|
| - : CompositorTimingHistory(NULL_UMA, rendering_stats), test_(test) {}
|
| + : CompositorTimingHistory(false, NULL_UMA, rendering_stats),
|
| + test_(test) {}
|
|
|
| protected:
|
| base::TimeTicks Now() const override;
|
| @@ -68,7 +69,7 @@ TEST_F(CompositorTimingHistoryTest, AllSequential_Commit) {
|
| base::TimeDelta activate_duration = base::TimeDelta::FromMilliseconds(4);
|
| base::TimeDelta draw_duration = base::TimeDelta::FromMilliseconds(5);
|
|
|
| - timing_history_.WillBeginMainFrame(true);
|
| + timing_history_.WillBeginMainFrame(true, Now());
|
| AdvanceNowBy(begin_main_frame_queue_duration);
|
| timing_history_.BeginMainFrameStarted(Now());
|
| AdvanceNowBy(begin_main_frame_start_to_commit_duration);
|
| @@ -87,7 +88,7 @@ TEST_F(CompositorTimingHistoryTest, AllSequential_Commit) {
|
| AdvanceNowBy(one_second);
|
| timing_history_.WillDraw();
|
| AdvanceNowBy(draw_duration);
|
| - timing_history_.DidDraw(true);
|
| + timing_history_.DidDraw(true, true, Now());
|
|
|
| EXPECT_EQ(begin_main_frame_queue_duration,
|
| timing_history_.BeginMainFrameQueueDurationCriticalEstimate());
|
| @@ -126,7 +127,7 @@ TEST_F(CompositorTimingHistoryTest, AllSequential_BeginMainFrameAborted) {
|
| base::TimeDelta activate_duration = base::TimeDelta::FromMilliseconds(4);
|
| base::TimeDelta draw_duration = base::TimeDelta::FromMilliseconds(5);
|
|
|
| - timing_history_.WillBeginMainFrame(true);
|
| + timing_history_.WillBeginMainFrame(false, Now());
|
| AdvanceNowBy(begin_main_frame_queue_duration);
|
| timing_history_.BeginMainFrameStarted(Now());
|
| AdvanceNowBy(begin_main_frame_start_to_commit_duration);
|
| @@ -146,9 +147,9 @@ TEST_F(CompositorTimingHistoryTest, AllSequential_BeginMainFrameAborted) {
|
| AdvanceNowBy(one_second);
|
| timing_history_.WillDraw();
|
| AdvanceNowBy(draw_duration);
|
| - timing_history_.DidDraw(true);
|
| + timing_history_.DidDraw(false, false, Now());
|
|
|
| - EXPECT_EQ(begin_main_frame_queue_duration,
|
| + EXPECT_EQ(base::TimeDelta(),
|
| timing_history_.BeginMainFrameQueueDurationCriticalEstimate());
|
| EXPECT_EQ(begin_main_frame_queue_duration,
|
| timing_history_.BeginMainFrameQueueDurationNotCriticalEstimate());
|
| @@ -179,13 +180,13 @@ TEST_F(CompositorTimingHistoryTest, BeginMainFrame_CriticalFaster) {
|
| base::TimeDelta begin_main_frame_start_to_commit_duration =
|
| base::TimeDelta::FromMilliseconds(1);
|
|
|
| - timing_history_.WillBeginMainFrame(true);
|
| + timing_history_.WillBeginMainFrame(true, Now());
|
| AdvanceNowBy(begin_main_frame_queue_duration_critical);
|
| timing_history_.BeginMainFrameStarted(Now());
|
| AdvanceNowBy(begin_main_frame_start_to_commit_duration);
|
| timing_history_.BeginMainFrameAborted();
|
|
|
| - timing_history_.WillBeginMainFrame(false);
|
| + timing_history_.WillBeginMainFrame(false, Now());
|
| AdvanceNowBy(begin_main_frame_queue_duration_not_critical);
|
| timing_history_.BeginMainFrameStarted(Now());
|
| AdvanceNowBy(begin_main_frame_start_to_commit_duration);
|
| @@ -219,7 +220,7 @@ TEST_F(CompositorTimingHistoryTest, BeginMainFrames_OldCriticalSlower) {
|
| base::TimeDelta::FromMilliseconds(1);
|
|
|
| // A single critical frame that is slow.
|
| - timing_history_.WillBeginMainFrame(true);
|
| + timing_history_.WillBeginMainFrame(true, Now());
|
| AdvanceNowBy(begin_main_frame_queue_duration_critical);
|
| timing_history_.BeginMainFrameStarted(Now());
|
| AdvanceNowBy(begin_main_frame_start_to_commit_duration);
|
| @@ -228,7 +229,7 @@ TEST_F(CompositorTimingHistoryTest, BeginMainFrames_OldCriticalSlower) {
|
|
|
| // A bunch of faster non critical frames that are newer.
|
| for (int i = 0; i < 100; i++) {
|
| - timing_history_.WillBeginMainFrame(false);
|
| + timing_history_.WillBeginMainFrame(false, Now());
|
| AdvanceNowBy(begin_main_frame_queue_duration_not_critical);
|
| timing_history_.BeginMainFrameStarted(Now());
|
| AdvanceNowBy(begin_main_frame_start_to_commit_duration);
|
| @@ -265,7 +266,7 @@ TEST_F(CompositorTimingHistoryTest, BeginMainFrames_NewCriticalSlower) {
|
| base::TimeDelta::FromMilliseconds(1);
|
|
|
| // A single non critical frame that is fast.
|
| - timing_history_.WillBeginMainFrame(false);
|
| + timing_history_.WillBeginMainFrame(false, Now());
|
| AdvanceNowBy(begin_main_frame_queue_duration_not_critical);
|
| timing_history_.BeginMainFrameStarted(Now());
|
| AdvanceNowBy(begin_main_frame_start_to_commit_duration);
|
| @@ -273,7 +274,7 @@ TEST_F(CompositorTimingHistoryTest, BeginMainFrames_NewCriticalSlower) {
|
|
|
| // A bunch of slower critical frames that are newer.
|
| for (int i = 0; i < 100; i++) {
|
| - timing_history_.WillBeginMainFrame(true);
|
| + timing_history_.WillBeginMainFrame(true, Now());
|
| AdvanceNowBy(begin_main_frame_queue_duration_critical);
|
| timing_history_.BeginMainFrameStarted(Now());
|
| AdvanceNowBy(begin_main_frame_start_to_commit_duration);
|
|
|