cc/blimp: Add synchronization for scroll/scale state.

cc/blimp/compositor_state_deserializer.h

Index: cc/blimp/compositor_state_deserializer.h
diff --git a/cc/blimp/compositor_state_deserializer.h b/cc/blimp/compositor_state_deserializer.h
index 141e341a122dff5943103190eabcac995b653dce..71e5fec1561fe61af6ab1e3648ba201a59aa915f 100644
--- a/cc/blimp/compositor_state_deserializer.h
+++ b/cc/blimp/compositor_state_deserializer.h
@@ -9,10 +9,14 @@
 
 #include "base/callback.h"
 #include "base/macros.h"
+#include "base/memory/weak_ptr.h"
 #include "cc/base/cc_export.h"
+#include "cc/blimp/synced_property_remote.h"
+#include "ui/gfx/geometry/scroll_offset.h"
 
 namespace cc {
 namespace proto {
+class ClientStateUpdate;
 class LayerNode;
 class LayerProperties;
 class LayerTree;
@@ -20,11 +24,20 @@ class LayerTreeHost;
 }  // namespace proto
 
 class ClientPictureCache;
-class CompositorStateDeserializerClient;
 class DeserializedContentLayerClient;
 class Layer;
 class LayerFactory;
-class LayerTreeHost;
+class LayerTreeHostInProcess;
+
+class CC_EXPORT CompositorStateDeserializerClient {
+ public:
+  virtual ~CompositorStateDeserializerClient() {}
+
+  // Used to inform the client that the local state received from the engine was
+  // modified on the impl thread and a ClientStateUpdate needs to be scheduled
+  // to synchronize it with the main thread on the engine.
+  virtual void DidUpdateLocalState() = 0;
+};
 
 // Deserializes the compositor updates into the LayerTreeHost.
 class CC_EXPORT CompositorStateDeserializer {
@@ -34,18 +47,14 @@ class CC_EXPORT CompositorStateDeserializer {
   using ScrollCallback = base::Callback<void(int engine_layer_id)>;

[kuzminruslan, 2016/10/30 13:18:41]

Perhaps not needed

[Khushal, 2016/10/31 23:00:27]

Thanks! Done.

 
   CompositorStateDeserializer(
-      LayerTreeHost* layer_tree_host,
+      LayerTreeHostInProcess* layer_tree_host,
       std::unique_ptr<ClientPictureCache> client_picture_cache,
-      const ScrollCallback& scroll_callback,
       CompositorStateDeserializerClient* client);
   ~CompositorStateDeserializer();
 
   // Returns the local layer on the client for the given |engine_layer_id|.
   Layer* GetLayerForEngineId(int engine_layer_id) const;
 
-  // Returns the local layer id on the client for the given |engine_layer_id|.
-  int GetClientIdFromEngineId(int engine_layer_id) const;
-
   // Deserializes the |layer_tree_host_proto| into the LayerTreeHost.
   void DeserializeCompositorUpdate(
       const proto::LayerTreeHost& layer_tree_host_proto);
@@ -53,7 +62,36 @@ class CC_EXPORT CompositorStateDeserializer {
   // Allows tests to inject the LayerFactory.
   void SetLayerFactoryForTesting(std::unique_ptr<LayerFactory> layer_factory);
 
+  // Updates any viewport related deltas that have been reported to the main
+  // thread from the impl thread.
+  void ApplyViewportDeltas(const gfx::Vector2dF& inner_delta,
+                           const gfx::Vector2dF& outer_delta,
+                           const gfx::Vector2dF& elastic_overscroll_delta,
+                           float page_scale,
+                           float top_controls_delta);
+
+  // Pulls a state update for changes reported to the main thread, that need to
+  // be synchronized with the associated state on the engine main thread.
+  void PullClientStateUpdate(proto::ClientStateUpdate* client_state_update);
+
+  // Informs that the client state update pulled was applied to the main thread
+  // on the engine.
+  // Note that this assumes that the state update provided to the engine was
+  // reflected back by the engine. If the application of this update resulted in
+  // any changes to the main thread state on the engine, these must be
+  // de-serialized and applied to the LayerTreeHost before a frame is committed
+  // to the impl thread.
+  void DidApplyStateUpdatesOnEngine();
+
+  // Sends any deltas that have been received on the main thread, but have not
+  // yet been applied to the main thread state back to the impl thread. This
+  // must be called for every main frame sent to the impl thread.
+  void SendUnappliedDeltasToLayerTreeHost();
+
  private:
+  using SyncedRemoteScrollOffset =
+      SyncedPropertyRemote<AdditionGroup<gfx::ScrollOffset>>;
+
   // A holder for the Layer and any data tied to it.
   struct LayerData {
     LayerData();
@@ -67,6 +105,8 @@ class CC_EXPORT CompositorStateDeserializer {
     // Set only for PictureLayers.
     std::unique_ptr<DeserializedContentLayerClient> content_layer_client;
 
+    SyncedRemoteScrollOffset synced_scroll_offset;
+
    private:
     DISALLOW_COPY_AND_ASSIGN(LayerData);
   };
@@ -89,18 +129,24 @@ class CC_EXPORT CompositorStateDeserializer {
       EngineIdToLayerMap* new_layer_map,
       ScrollbarLayerToScrollLayerId* scrollbar_layer_to_scroll_layer);
 
+  void LayerScrolled(int engine_layer_id);
+
   scoped_refptr<Layer> GetLayer(int engine_layer_id) const;
   DeserializedContentLayerClient* GetContentLayerClient(
       int engine_layer_id) const;
+  int GetClientIdFromEngineId(int engine_layer_id) const;
+  LayerData* GetLayerData(int engine_layer_id);
 
   std::unique_ptr<LayerFactory> layer_factory_;
 
-  LayerTreeHost* layer_tree_host_;
+  LayerTreeHostInProcess* layer_tree_host_;
   std::unique_ptr<ClientPictureCache> client_picture_cache_;
-  ScrollCallback scroll_callback_;
   CompositorStateDeserializerClient* client_;
 
   EngineIdToLayerMap engine_id_to_layer_;
+  SyncedPropertyRemote<ScaleGroup> synced_page_scale_;
+
+  base::WeakPtrFactory<CompositorStateDeserializer> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(CompositorStateDeserializer);
 };