Implemented GetWindowSnapshot on RenderViewImpl

content/common/view_messages.h

 // Copyright (c) 2012 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.
 
 // IPC messages for page rendering.
 // Multiply-included message file, hence no include guard.
 
 #include "base/process.h"
 #include "base/shared_memory.h"
 #include "content/common/content_export.h"
@@ skipping 1914 matching lines @@
 IPC_MESSAGE_ROUTED1(ViewHostMsg_HasTouchEventHandlers,
                     bool /* has_handlers */)
 
 // A message from HTML-based UI.  When (trusted) Javascript calls
 // send(message, args), this message is sent to the browser.
 IPC_MESSAGE_ROUTED3(ViewHostMsg_WebUISend,
                     GURL /* source_url */,
                     std::string  /* message */,
                     base::ListValue /* args */)
 
+// Requests a snapshot of the given window

[Ken Russell (switch to Gerrit), 2012/11/08 22:27:44]

Add "." here and below.

+IPC_MESSAGE_ROUTED1(ViewHostMsg_GetWindowSnapshot,
+                    int      /* message id */)

[jam, 2012/11/09 02:21:29]

nit: here and below, just one space before start of comment per convention

+
+// Notifies the render view that a snapshot has been retrieved
+IPC_MESSAGE_ROUTED3(ViewMsg_WindowSnapshotCompleted,
+                    int      /* message id */,
+                    gfx::Size /* image dimensions */,

[jam, 2012/11/09 02:21:29]

nit: here and in the rest of the lines you added, the convention is that the
parameter name in the dispatching function is used inside the comment

+                    std::vector<unsigned char> /* png data */)
+
 // A renderer sends this to the browser process when it wants to create a ppapi
 // plugin.  The browser will create the plugin process if necessary, and will
 // return a handle to the channel on success.
 //
 // The plugin_child_id is the ChildProcessHost ID assigned in the browser
 // process. This ID is valid only in the context of the browser process and is
 // used to identify the proper process when the renderer notifies it that the
 // plugin is hung.
 //
 // On error an empty string and null handles are returned.
@@ skipping 460 matching lines @@
 IPC_MESSAGE_ROUTED1(ViewHostMsg_StartContentIntent,
                     GURL /* content_url */)
 #endif
 
 // Notifies that multiple touch targets may have been pressed, and to show
 // the disambiguation popup.
 IPC_MESSAGE_ROUTED3(ViewHostMsg_ShowDisambiguationPopup,
                     gfx::Rect, /* Border of touched targets */
                     gfx::Size, /* Size of zoomed image */
                     TransportDIB::Id /* DIB of zoomed image */)