Fixing flakiness of TextureLayerChangeInvisibleMailboxTest

cc/test/test_hooks.h

 // Copyright 2015 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.
 
 #ifndef CC_TEST_TEST_HOOKS_H_
 #define CC_TEST_TEST_HOOKS_H_
 
 #include "base/macros.h"
 #include "cc/animation/animation_delegate.h"
 #include "cc/trees/layer_tree_host.h"
@@ skipping 43 matching lines @@
   virtual void DidSetVisibleOnImplTree(LayerTreeHostImpl* host_impl,
                                        bool visible) {}
   virtual void AnimateLayers(LayerTreeHostImpl* host_impl,
                              base::TimeTicks monotonic_time) {}
   virtual void UpdateAnimationState(LayerTreeHostImpl* host_impl,
                                     bool has_unfinished_animation) {}
   virtual void WillAnimateLayers(LayerTreeHostImpl* host_impl,
                                  base::TimeTicks monotonic_time) {}
   virtual void DidInvalidateContentOnImplSide(LayerTreeHostImpl* host_impl) {}
   virtual void DidRequestImplSideInvalidation(LayerTreeHostImpl* host_impl) {}
+  virtual void WillReceiveCompositorFrameAckOnImplThread() {}

[danakj, 2017/04/18 20:58:42]

just "OnThread" is the convention used here, and can you group it with the other
OnThread things just above?

[Saman Sami, 2017/04/18 21:37:13]

Done.

+  virtual void DidReceiveCompositorFrameAckOnImplThread() {}
 
   // Asynchronous compositor thread hooks.
   // These are called asynchronously from the LayerTreeHostImpl performing its
   // draw, so you should record state you want to use here in
   // DrawLayersOnThread() instead. For that reason these methods do not receive
   // a LayerTreeHostImpl pointer.
   virtual void DisplayReceivedLocalSurfaceIdOnThread(
       const LocalSurfaceId& local_surface_id) {}
   virtual void DisplayReceivedCompositorFrameOnThread(
       const CompositorFrame& frame) {}
@@ skipping 43 matching lines @@
   // OutputSurface indirections to the LayerTreeTest, that can be further
   // overridden.
   virtual void RequestNewCompositorFrameSink() = 0;
   virtual std::unique_ptr<OutputSurface> CreateDisplayOutputSurfaceOnThread(
       scoped_refptr<ContextProvider> compositor_context_provider) = 0;
 };
 
 }  // namespace cc
 
 #endif  // CC_TEST_TEST_HOOKS_H_