Migrate Telemetry from beginWindowSnapshotPNG to Page.captureScreenshot

content/browser/devtools/renderer_overrides_handler.h

Index: content/browser/devtools/renderer_overrides_handler.h
diff --git a/content/browser/devtools/renderer_overrides_handler.h b/content/browser/devtools/renderer_overrides_handler.h
index cce32767c39b7ba291ecff01bf926d54101a16a0..6b2006286df79ddce7597ed863706e5ad994fd53 100644
--- a/content/browser/devtools/renderer_overrides_handler.h
+++ b/content/browser/devtools/renderer_overrides_handler.h
@@ -75,6 +75,10 @@ class CONTENT_EXPORT RendererOverridesHandler
 
   void ScreenshotCaptured(
       scoped_refptr<DevToolsProtocol::Command> command,
+      const unsigned char* png_data,
+      size_t png_size);
+
+  void ScreencastFrameCaptured(

[Ken Russell (switch to Gerrit), 2014/03/11 18:22:29]

What is the difference between ScreenshotCaptured and ScreencastFrameCaptured?
It seems to me that only one of these should exist.

[Vladislav Kaznacheev, 2014/03/12 09:24:24]

These are two different things.
Screencast is a special mode when DevTools captures compositor frames as they
are generated and sends them to DevTools frontend. In this case there is no need
to force synchronization with the renderer.

On 2014/03/11 18:22:29, Ken Russell wrote:

       const std::string& format,
       int quality,
       const cc::CompositorFrameMetadata& metadata,