Android Webview IPC-based sync compositing

content/browser/android/synchronous_compositor_host.h

Index: content/browser/android/synchronous_compositor_host.h
diff --git a/content/browser/android/synchronous_compositor_host.h b/content/browser/android/synchronous_compositor_host.h
new file mode 100644
index 0000000000000000000000000000000000000000..ac30ddab0e57cd313200d4a7c5f9349b585a0fb3
--- /dev/null
+++ b/content/browser/android/synchronous_compositor_host.h
@@ -0,0 +1,81 @@
+// 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 CONTENT_BROWSER_ANDROID_SYNCHRONOUS_COMPOSITOR_HOST_H_
+#define CONTENT_BROWSER_ANDROID_SYNCHRONOUS_COMPOSITOR_HOST_H_
+
+#include "cc/output/compositor_frame.h"
+#include "content/browser/android/synchronous_compositor_base.h"
+#include "ui/gfx/geometry/scroll_offset.h"
+#include "ui/gfx/geometry/size_f.h"
+
+namespace IPC {
+class Sender;
+}
+
+namespace content {
+
+class RenderWidgetHostViewAndroid;
+class SynchronousCompositorClient;
+struct CommonBrowserParams;
+struct CommonRendererParams;
+
+class SynchronousCompositorHost : public SynchronousCompositorBase {
+ public:
+  ~SynchronousCompositorHost() override;
+
+  // SynchronousCompositor overrides.
+  scoped_ptr<cc::CompositorFrame> DemandDrawHw(
+      gfx::Size surface_size,

[dcheng, 2015/10/27 21:06:58]

gfx::Size and gfx::Rect are usually passed by const ref aren't they?

[boliu, 2015/10/28 02:07:06]

I _remember_ at some point these were supposed to be passed by value because
they are two ints. Ha, found this:
https://code.google.com/p/chromium/issues/detail?id=159273#c31, so const ref it
is then.

Pulling it out into https://codereview.chromium.org/1417893006/, and I'll rebase
this patch when that lands. For now, I'll keep this building on top of trunk.

+      const gfx::Transform& transform,
+      gfx::Rect viewport,
+      gfx::Rect clip,
+      gfx::Rect viewport_rect_for_tile_priority,
+      const gfx::Transform& transform_for_tile_priority) override;
+  bool DemandDrawSw(SkCanvas* canvas) override;
+  void ReturnResources(const cc::CompositorFrameAck& frame_ack) override;
+  void SetMemoryPolicy(size_t bytes_limit) override;
+  void DidChangeRootLayerScrollOffset(
+      const gfx::ScrollOffset& root_offset) override;
+  void SetIsActive(bool is_active) override;
+  void OnComputeScroll(base::TimeTicks animation_time) override;
+
+  // SynchronousCompositorBase overrides.
+  InputEventAckState HandleInputEvent(
+      const blink::WebInputEvent& input_event) override;
+  void BeginFrame(const cc::BeginFrameArgs& args) override;
+  bool OnMessageReceived(const IPC::Message& message) override;
+
+ private:
+  friend class SynchronousCompositorBase;
+  SynchronousCompositorHost(RenderWidgetHostViewAndroid* rwhva,
+                            SynchronousCompositorClient* client);
+  CommonBrowserParams ConstructCommonParams();
+  void ProcessCommonParams(const CommonRendererParams& params);
+  void UpdateNeedsBeginFrames();
+  void UpdateFrameMetaData(const cc::CompositorFrameMetadata& frame_metadata);
+
+  RenderWidgetHostViewAndroid* const rwhva_;
+  SynchronousCompositorClient* const client_;
+  const int routing_id_;
+  IPC::Sender* const sender_;
+
+  bool is_active_;
+  size_t bytes_limit_;
+  cc::ReturnedResourceArray returned_resources_;
+
+  // Updated by both renderer and browser.
+  gfx::ScrollOffset root_scroll_offset_;
+
+  // From renderer.
+  unsigned int renderer_param_version_;
+  bool need_animate_scroll_;
+  bool need_invalidate_;
+  bool need_begin_frame_;
+  bool did_activate_pending_tree_;
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_ANDROID_SYNCHRONOUS_COMPOSITOR_HOST_H_