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

Unified Diff: cc/scheduler/frame_rate_controller_unittest.cc

Issue 15058004: cc: Rename VSync to BeginFrame (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Fix test compiler errors Created 7 years, 7 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
Index: cc/scheduler/frame_rate_controller_unittest.cc
diff --git a/cc/scheduler/frame_rate_controller_unittest.cc b/cc/scheduler/frame_rate_controller_unittest.cc
index 0ed600a02df00ec7a9f22864768f0d3dad39e3f7..6bb888ef230650747dbf0fcdb08520dbdb3a2c97 100644
--- a/cc/scheduler/frame_rate_controller_unittest.cc
+++ b/cc/scheduler/frame_rate_controller_unittest.cc
@@ -14,15 +14,15 @@ class FakeFrameRateControllerClient : public cc::FrameRateControllerClient {
public:
FakeFrameRateControllerClient() { Reset(); }
- void Reset() { vsync_ticked_ = false; }
- bool VSyncTicked() const { return vsync_ticked_; }
+ void Reset() { began_frame_ = false; }
+ bool BeganFrame() const { return began_frame_; }
- virtual void VSyncTick(bool throttled) OVERRIDE {
- vsync_ticked_ = !throttled;
+ virtual void BeginFrame(bool throttled) OVERRIDE {
+ began_frame_ = !throttled;
}
protected:
- bool vsync_ticked_;
+ bool began_frame_;
};
TEST(FrameRateControllerTest, TestFrameThrottling_ImmediateAck) {
@@ -39,11 +39,11 @@ TEST(FrameRateControllerTest, TestFrameThrottling_ImmediateAck) {
base::TimeTicks elapsed; // Muck around with time a bit
- // Trigger one frame, make sure the vsync callback is called
+ // Trigger one frame, make sure the BeginFrame callback is called
elapsed += base::TimeDelta::FromMilliseconds(thread.PendingDelayMs());
time_source->SetNow(elapsed);
thread.RunPendingTask();
- EXPECT_TRUE(client.VSyncTicked());
+ EXPECT_TRUE(client.BeganFrame());
client.Reset();
// Tell the controller we drew
@@ -54,13 +54,13 @@ TEST(FrameRateControllerTest, TestFrameThrottling_ImmediateAck) {
base::TimeDelta::FromMilliseconds(5));
controller.DidFinishFrame();
- // Trigger another frame, make sure vsync runs again
+ // Trigger another frame, make sure BeginFrame runs again
elapsed += base::TimeDelta::FromMilliseconds(thread.PendingDelayMs());
// Sanity check that previous code didn't move time backward.
EXPECT_GE(elapsed, time_source->Now());
time_source->SetNow(elapsed);
thread.RunPendingTask();
- EXPECT_TRUE(client.VSyncTicked());
+ EXPECT_TRUE(client.BeganFrame());
}
TEST(FrameRateControllerTest, TestFrameThrottling_TwoFramesInFlight) {
@@ -78,23 +78,23 @@ TEST(FrameRateControllerTest, TestFrameThrottling_TwoFramesInFlight) {
base::TimeTicks elapsed; // Muck around with time a bit
- // Trigger one frame, make sure the vsync callback is called
+ // Trigger one frame, make sure the BeginFrame callback is called
elapsed += base::TimeDelta::FromMilliseconds(thread.PendingDelayMs());
time_source->SetNow(elapsed);
thread.RunPendingTask();
- EXPECT_TRUE(client.VSyncTicked());
+ EXPECT_TRUE(client.BeganFrame());
client.Reset();
// Tell the controller we drew
controller.DidBeginFrame();
- // Trigger another frame, make sure vsync callback runs again
+ // Trigger another frame, make sure BeginFrame callback runs again
elapsed += base::TimeDelta::FromMilliseconds(thread.PendingDelayMs());
// Sanity check that previous code didn't move time backward.
EXPECT_GE(elapsed, time_source->Now());
time_source->SetNow(elapsed);
thread.RunPendingTask();
- EXPECT_TRUE(client.VSyncTicked());
+ EXPECT_TRUE(client.BeganFrame());
client.Reset();
// Tell the controller we drew, again.
@@ -106,7 +106,7 @@ TEST(FrameRateControllerTest, TestFrameThrottling_TwoFramesInFlight) {
EXPECT_GE(elapsed, time_source->Now());
time_source->SetNow(elapsed);
thread.RunPendingTask();
- EXPECT_FALSE(client.VSyncTicked());
+ EXPECT_FALSE(client.BeganFrame());
// Tell the controller the first frame ended 5ms later
time_source->SetNow(time_source->Now() +
@@ -114,16 +114,16 @@ TEST(FrameRateControllerTest, TestFrameThrottling_TwoFramesInFlight) {
controller.DidFinishFrame();
// Tick should not have been called
- EXPECT_FALSE(client.VSyncTicked());
+ EXPECT_FALSE(client.BeganFrame());
- // Trigger yet another frame. Since one frames is pending, another vsync
+ // Trigger yet another frame. Since one frames is pending, another BeginFrame
// callback should run.
elapsed += base::TimeDelta::FromMilliseconds(thread.PendingDelayMs());
// Sanity check that previous code didn't move time backward.
EXPECT_GE(elapsed, time_source->Now());
time_source->SetNow(elapsed);
thread.RunPendingTask();
- EXPECT_TRUE(client.VSyncTicked());
+ EXPECT_TRUE(client.BeganFrame());
}
TEST(FrameRateControllerTest, TestFrameThrottling_Unthrottled) {
@@ -134,44 +134,46 @@ TEST(FrameRateControllerTest, TestFrameThrottling_Unthrottled) {
controller.SetClient(&client);
controller.SetMaxFramesPending(2);
- // SetActive triggers 1st frame, make sure the vsync callback is called
+ // SetActive triggers 1st frame, make sure the BeginFrame callback is called
controller.SetActive(true);
thread.RunPendingTask();
- EXPECT_TRUE(client.VSyncTicked());
+ EXPECT_TRUE(client.BeganFrame());
client.Reset();
// Even if we don't call DidBeginFrame, FrameRateController should
- // still attempt to vsync tick multiple times until it does result in
+ // still attempt to tick multiple times until it does result in
// a DidBeginFrame.
thread.RunPendingTask();
- EXPECT_TRUE(client.VSyncTicked());
+ EXPECT_TRUE(client.BeganFrame());
client.Reset();
thread.RunPendingTask();
- EXPECT_TRUE(client.VSyncTicked());
+ EXPECT_TRUE(client.BeganFrame());
client.Reset();
- // DidBeginFrame triggers 2nd frame, make sure the vsync callback is called
+ // DidBeginFrame triggers 2nd frame, make sure the BeginFrame callback is
+ // called
controller.DidBeginFrame();
thread.RunPendingTask();
- EXPECT_TRUE(client.VSyncTicked());
+ EXPECT_TRUE(client.BeganFrame());
client.Reset();
// DidBeginFrame triggers 3rd frame (> max_frames_pending),
- // make sure the vsync callback is NOT called
+ // make sure the BeginFrame callback is NOT called
controller.DidBeginFrame();
thread.RunPendingTask();
- EXPECT_FALSE(client.VSyncTicked());
+ EXPECT_FALSE(client.BeganFrame());
client.Reset();
// Make sure there is no pending task since we can't do anything until we
// receive a DidFinishFrame anyway.
EXPECT_FALSE(thread.HasPendingTask());
- // DidFinishFrame triggers a frame, make sure the vsync callback is called
+ // DidFinishFrame triggers a frame, make sure the BeginFrame callback is
+ // called
controller.DidFinishFrame();
thread.RunPendingTask();
- EXPECT_TRUE(client.VSyncTicked());
+ EXPECT_TRUE(client.BeganFrame());
}
} // namespace

Powered by Google App Engine
This is Rietveld 408576698