Remove RenderWidgetHost::GetSnapshotFromRenderer.

chrome/browser/extensions/api/capture_web_contents_function.cc

 // Copyright 2013 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.
 
 #include "chrome/browser/extensions/api/capture_web_contents_function.h"
 
 #include "base/base64.h"
 #include "base/strings/stringprintf.h"
 #include "chrome/browser/extensions/api/tabs/tabs_constants.h"
 #include "content/public/browser/render_view_host.h"
@@ skipping 71 matching lines @@
 }
 
 void CaptureWebContentsFunction::CopyFromBackingStoreComplete(
     bool succeeded,
     const SkBitmap& bitmap) {
   if (succeeded) {
     OnCaptureSuccess(bitmap);
     return;
   }
 
   WebContents* contents = GetWebContentsForID(context_id_);

[Lei Zhang, 2014/04/24 23:50:43]

Do you need this anymore? The only reason we looked up |contents| is to use it
below.

[danakj, 2014/04/25 15:20:16]

Done.

   if (!contents) {
     OnCaptureFailure(FAILURE_REASON_CONTENT_NOT_FOUND);
     return;
   }
 
-  // Ask the renderer for a snapshot of the page.
-  RenderWidgetHost* render_widget_host = contents->GetRenderViewHost();
-  render_widget_host->GetSnapshotFromRenderer(
-      gfx::Rect(),
-      base::Bind(&CaptureWebContentsFunction::GetSnapshotFromRendererComplete,
-                 this));
-}
-
-void CaptureWebContentsFunction::GetSnapshotFromRendererComplete(
-    bool succeeded,
-    const SkBitmap& bitmap) {
-  if (succeeded)
-    OnCaptureSuccess(bitmap);
-  else
-    OnCaptureFailure(FAILURE_REASON_UNKNOWN);
+  OnCaptureFailure(FAILURE_REASON_UNKNOWN);
 }
 
 void CaptureWebContentsFunction::OnCaptureSuccess(const SkBitmap& bitmap) {
   std::vector<unsigned char> data;
   SkAutoLockPixels screen_capture_lock(bitmap);
   bool encoded = false;
   std::string mime_type;
   switch (image_format_) {
     case ImageDetails::FORMAT_JPEG:
       encoded = gfx::JPEGCodec::Encode(
@@ skipping 27 matching lines @@
       reinterpret_cast<const char*>(vector_as_array(&data)), data.size());
 
   base::Base64Encode(stream_as_string, &base64_result);
   base64_result.insert(0, base::StringPrintf("data:%s;base64,",
                                              mime_type.c_str()));
   SetResult(new base::StringValue(base64_result));
   SendResponse(true);
 }
 
 }  // namespace extensions