Factor FrameData out of RendererCompositorFrameSink

content/common/render_widget_surface_properties.h

Index: content/common/render_widget_surface_properties.h
diff --git a/content/common/render_widget_surface_properties.h b/content/common/render_widget_surface_properties.h
new file mode 100644
index 0000000000000000000000000000000000000000..0007f01769b16edca7b49e356602370985f74016
--- /dev/null
+++ b/content/common/render_widget_surface_properties.h
@@ -0,0 +1,40 @@
+// Copyright 2017 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_COMMON_RENDER_WIDGET_SURFACE_PROPERTIES_H_
+#define CONTENT_COMMON_RENDER_WIDGET_SURFACE_PROPERTIES_H_
+
+#include "cc/output/compositor_frame.h"
+#include "content/common/content_export.h"
+
+namespace content {
+
+// This struct contains the properties that are constant among all
+// CompositorFrames that the renderer submits to the same surface.
+struct CONTENT_EXPORT RenderWidgetSurfaceProperties {
+  static RenderWidgetSurfaceProperties FromCompositorFrame(
+      const cc::CompositorFrame& frame);
+
+  RenderWidgetSurfaceProperties();
+  RenderWidgetSurfaceProperties(const RenderWidgetSurfaceProperties& other);

[piman, 2017/05/17 15:14:06]

nit: also add operator =(const RenderWidgetSurfaceProperties& other)
(rule-of-3)

[Saman Sami, 2017/05/17 15:38:52]

Done.

+  ~RenderWidgetSurfaceProperties();
+
+  bool operator==(const RenderWidgetSurfaceProperties& other) const;
+  bool operator!=(const RenderWidgetSurfaceProperties& other) const;
+
+  gfx::Size size;
+  float device_scale_factor = 0;
+#ifdef OS_ANDROID
+  float top_controls_height = 0;
+  float top_controls_shown_ratio = 0;
+  float bottom_controls_height = 0;
+  float bottom_controls_shown_ratio = 0;
+  cc::Selection<gfx::SelectionBound> selection;
+  bool has_transparent_background = false;
+#endif
+};
+
+}  // namespace content
+
+#endif  // CONTENT_COMMON_RENDER_WIDGET_SURFACE_PROPERTIES_H_