Add fromSurface optional parameter to devtools Page.CaptureScreenshot

content/browser/renderer_host/render_widget_host_impl.h

Index: content/browser/renderer_host/render_widget_host_impl.h
diff --git a/content/browser/renderer_host/render_widget_host_impl.h b/content/browser/renderer_host/render_widget_host_impl.h
index 11dddf13c0f1ebfe7a95f53baaa4490f5c19f65c..21454cb53ae83c9963ee464096ec9c95844dffef 100644
--- a/content/browser/renderer_host/render_widget_host_impl.h
+++ b/content/browser/renderer_host/render_widget_host_impl.h
@@ -49,6 +49,7 @@
 #include "ui/events/latency_info.h"
 #include "ui/gfx/native_widget_types.h"
 
+class SkBitmap;
 struct FrameHostMsg_HittestData_Params;
 struct ViewHostMsg_SelectionBounds_Params;
 struct ViewHostMsg_UpdateRect_Params;
@@ -208,6 +209,8 @@ class CONTENT_EXPORT RenderWidgetHostImpl : public RenderWidgetHost,
   using GetSnapshotFromBrowserCallback =
       base::Callback<void(const gfx::Image&)>;
   void GetSnapshotFromBrowser(const GetSnapshotFromBrowserCallback& callback);
+  void GetSurfaceSnapshotFromBrowser(
+      const GetSnapshotFromBrowserCallback& callback);
 
   const NativeWebKeyboardEvent* GetLastKeyboardEvent() const;
 
@@ -695,6 +698,11 @@ class CONTENT_EXPORT RenderWidgetHostImpl : public RenderWidgetHost,
 
   void WindowSnapshotReachedScreen(int snapshot_id);
 
+  void OnSnapshotFromSurfaceReceived(int snapshot_id,
+                                     int retry_count,
+                                     const SkBitmap& bitmap,
+                                     ReadbackResponse response);
+
   void OnSnapshotReceived(int snapshot_id, const gfx::Image& image);
 
   // 1. Grants permissions to URL (if any)
@@ -859,6 +867,10 @@ class CONTENT_EXPORT RenderWidgetHostImpl : public RenderWidgetHost,
   using PendingSnapshotMap = std::map<int, GetSnapshotFromBrowserCallback>;
   PendingSnapshotMap pending_browser_snapshots_;
 
+  using PendingSnapshotFromSurfaceMap =
+      std::map<int, GetSnapshotFromBrowserCallback>;
+  PendingSnapshotFromSurfaceMap pending_surface_browser_snapshots_;

[Eric Seckler, 2017/03/09 09:55:22]

nit: could be PendingSnapshotMap now, too.

[dvallet, 2017/03/09 22:43:52]

Done.

+
   // Indicates whether a RenderFramehost has ownership, in which case this
   // object does not self destroy.
   bool owned_by_render_frame_host_;