Refactor delegated scrolling in LayerImpl

content/browser/android/in_process/synchronous_compositor_impl.h

Index: content/browser/android/in_process/synchronous_compositor_impl.h
diff --git a/content/browser/android/in_process/synchronous_compositor_impl.h b/content/browser/android/in_process/synchronous_compositor_impl.h
index d6cacf0dee66e8e0865d2695d988d97dc4dbd4c6..f589d18c023099a3ef859ab951ca81e76b93401b 100644
--- a/content/browser/android/in_process/synchronous_compositor_impl.h
+++ b/content/browser/android/in_process/synchronous_compositor_impl.h
@@ -82,6 +82,7 @@ class SynchronousCompositorImpl
                             float min_page_scale_factor,
                             float max_page_scale_factor) override;
   bool IsExternalFlingActive() const override;
+  void SetBlockUpdates(bool block_updates) override;
 
   void SetInputHandler(cc::InputHandler* input_handler);
   void DidOverscroll(const DidOverscrollParams& params);
@@ -107,6 +108,7 @@ class SynchronousCompositorImpl
   const int routing_id_;
   cc::InputHandler* input_handler_;
   bool invoking_composite_;
+  bool block_updates_;

[aelias_OOO_until_Jul13, 2015/03/30 22:54:39]

Nobody ever reads this value, is there some other missing piece to this change?

Also, I don't really like the idea of a "block updates" boolean, we shouldn't
really need a bit of extra state just to maintain consistency on a single
thread.

[hush (inactive), 2015/03/30 23:00:44]

sorry I accidentally removed the usage in one of the patches. It should be used
by UpdateRootLayerState. I just added it back.
I will think about another way to achieve this.

 
   base::WeakPtrFactory<SynchronousCompositorImpl> weak_ptr_factory_;