Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3540)

Unified Diff: cc/blimp/remote_compositor_bridge_client.h

Issue 2445093002: cc/blimp: Add synchronization for scroll/scale state. (Closed)
Patch Set: test compile Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « cc/blimp/layer_tree_host_remote_unittest.cc ('k') | cc/blimp/synced_property_remote.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/blimp/remote_compositor_bridge_client.h
diff --git a/cc/blimp/remote_compositor_bridge_client.h b/cc/blimp/remote_compositor_bridge_client.h
index b167182ef7bd22208d4bf9bd91ae14c7de234d25..6f83ed541633cacd56c8e88e0d21065b645335d5 100644
--- a/cc/blimp/remote_compositor_bridge_client.h
+++ b/cc/blimp/remote_compositor_bridge_client.h
@@ -15,6 +15,9 @@ class ScrollOffset;
} // namespace gfx
namespace cc {
+namespace proto {
+class ClientStateUpdate;
+} // namespace proto
class CompositorProtoState;
class CC_EXPORT RemoteCompositorBridgeClient {
@@ -29,13 +32,10 @@ class CC_EXPORT RemoteCompositorBridgeClient {
// is made.
virtual void BeginMainFrame() = 0;
- // Provides an update from the mutations made on the client. Returns true if
- // the update could be successfully applied to the engine state. This can
- // fail, for instance, if the layer present in the update was destroyed on the
- // engine.
- virtual bool ApplyScrollAndScaleUpdateFromClient(
- const ScrollOffsetMap& client_scroll_map,
- float client_page_scale) = 0;
+ // Applied state updates reported from the client onto the main thread state
+ // on the engine.
+ virtual void ApplyStateUpdateFromClient(
+ const proto::ClientStateUpdate& client_state_update) = 0;
};
} // namespace cc
« no previous file with comments | « cc/blimp/layer_tree_host_remote_unittest.cc ('k') | cc/blimp/synced_property_remote.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698