Create a new action triggered when a BeginMainFrame is not expected before vsync

cc/test/layer_tree_test.cc

 // Copyright 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/test/layer_tree_test.h"
 
 #include "base/command_line.h"
 #include "base/location.h"
 #include "base/memory/ptr_util.h"
 #include "base/run_loop.h"
@@ skipping 336 matching lines @@
     test_hooks_->DidReceiveCompositorFrameAck();
   }
 
   void DidSubmitCompositorFrame() override {}
   void DidLoseCompositorFrameSink() override {}
   void RequestScheduleComposite() override { test_hooks_->ScheduleComposite(); }
   void DidCompletePageScaleAnimation() override {}
   void BeginMainFrameNotExpectedSoon() override {
     test_hooks_->BeginMainFrameNotExpectedSoon();
   }
+  void DoShortIdleWork(const BeginFrameArgs& args) override {
+    // TODO(delphick): do something here?

[Sami, 2017/03/24 15:24:11]

Unless we're testing something at this level this can just be blank.

[Dan Elphick, 2017/03/29 16:02:44]

Yeah it was just a reminder to check if we needed a test here since
BeginMainFrameNotExpectedSoon has hooks here. For now I'll remove it.

+  }
 
  private:
   explicit LayerTreeHostClientForTesting(TestHooks* test_hooks)
       : test_hooks_(test_hooks) {}
 
   TestHooks* test_hooks_;
 };
 
 // Adapts LayerTreeHost for test. Injects LayerTreeHostImplForTesting.
 class LayerTreeHostForTesting : public LayerTreeHost {
@@ skipping 519 matching lines @@
   DCHECK(task_runner_provider()->IsMainThread() ||
          task_runner_provider()->IsMainThreadBlocked());
   return layer_tree_host_.get();
 }
 
 Proxy* LayerTreeTest::proxy() {
   return layer_tree_host() ? layer_tree_host()->proxy() : NULL;
 }
 
 }  // namespace cc