| Index: cc/output/output_surface_unittest.cc
|
| diff --git a/cc/output/output_surface_unittest.cc b/cc/output/output_surface_unittest.cc
|
| index a73b7790cf30f897286efdc05bfc35680a85b836..beb6a0046866ac49854a4ef3d66240cd1d106c9c 100644
|
| --- a/cc/output/output_surface_unittest.cc
|
| +++ b/cc/output/output_surface_unittest.cc
|
| @@ -17,15 +17,18 @@ namespace {
|
| class TestOutputSurface : public OutputSurface {
|
| public:
|
| explicit TestOutputSurface(scoped_ptr<WebKit::WebGraphicsContext3D> context3d)
|
| - : OutputSurface(context3d.Pass()) {}
|
| + : OutputSurface(context3d.Pass()),
|
| + retroactive_begin_frame_deadline_enabled_(false) {}
|
|
|
| explicit TestOutputSurface(
|
| scoped_ptr<cc::SoftwareOutputDevice> software_device)
|
| - : OutputSurface(software_device.Pass()) {}
|
| + : OutputSurface(software_device.Pass()),
|
| + retroactive_begin_frame_deadline_enabled_(false) {}
|
|
|
| TestOutputSurface(scoped_ptr<WebKit::WebGraphicsContext3D> context3d,
|
| scoped_ptr<cc::SoftwareOutputDevice> software_device)
|
| - : OutputSurface(context3d.Pass(), software_device.Pass()) {}
|
| + : OutputSurface(context3d.Pass(), software_device.Pass()),
|
| + retroactive_begin_frame_deadline_enabled_(false) {}
|
|
|
| bool InitializeNewContext3D(
|
| scoped_ptr<WebKit::WebGraphicsContext3D> new_context3d) {
|
| @@ -58,8 +61,8 @@ class TestOutputSurface : public OutputSurface {
|
| OnSwapBuffersComplete(NULL);
|
| }
|
|
|
| - void SetRetroactiveBeginFramePeriod(base::TimeDelta period) {
|
| - retroactive_begin_frame_period_ = period;
|
| + void EnableRetroactiveBeginFrameDeadline(bool enable) {
|
| + retroactive_begin_frame_deadline_enabled_ = enable;
|
| }
|
|
|
| protected:
|
| @@ -68,11 +71,13 @@ class TestOutputSurface : public OutputSurface {
|
| CheckForRetroactiveBeginFrame();
|
| }
|
|
|
| - virtual base::TimeDelta RetroactiveBeginFramePeriod() OVERRIDE {
|
| - return retroactive_begin_frame_period_;
|
| + virtual base::TimeTicks RetroactiveBeginFrameDeadline() OVERRIDE {
|
| + if (retroactive_begin_frame_deadline_enabled_)
|
| + return OutputSurface::RetroactiveBeginFrameDeadline();
|
| + return base::TimeTicks();
|
| }
|
|
|
| - base::TimeDelta retroactive_begin_frame_period_;
|
| + bool retroactive_begin_frame_deadline_enabled_;
|
| };
|
|
|
| class FakeOutputSurfaceClient : public OutputSurfaceClient {
|
| @@ -237,8 +242,7 @@ TEST(OutputSurfaceTest, BeginFrameEmulation) {
|
| display_refresh_interval);
|
|
|
| output_surface.SetMaxFramesPending(2);
|
| - output_surface.SetRetroactiveBeginFramePeriod(
|
| - base::TimeDelta::FromSeconds(-1));
|
| + output_surface.EnableRetroactiveBeginFrameDeadline(false);
|
|
|
| // We should start off with 0 BeginFrames
|
| EXPECT_EQ(client.begin_frame_count(), 0);
|
| @@ -312,10 +316,7 @@ TEST(OutputSurfaceTest, OptimisticAndRetroactiveBeginFrames) {
|
| EXPECT_FALSE(client.deferred_initialize_called());
|
|
|
| output_surface.SetMaxFramesPending(2);
|
| -
|
| - // Enable retroactive BeginFrames.
|
| - output_surface.SetRetroactiveBeginFramePeriod(
|
| - base::TimeDelta::FromSeconds(100000));
|
| + output_surface.EnableRetroactiveBeginFrameDeadline(true);
|
|
|
| // Optimistically injected BeginFrames should be throttled if
|
| // SetNeedsBeginFrame is false...
|
|
|