Query the preferred readback config in CopyFromBackingStore

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 "chrome/common/extensions/extension_constants.h"
@@ skipping 54 matching lines @@
     if (image_details->quality.get())
       image_quality_ = *image_details->quality;
   }
 
   RenderViewHost* render_view_host = contents->GetRenderViewHost();
   RenderWidgetHostView* view = render_view_host->GetView();
   if (!view) {
     OnCaptureFailure(FAILURE_REASON_VIEW_INVISIBLE);
     return false;
   }
+  SkBitmap::Config preferred_format =
+      render_view_host->PreferredReadbackFormat();
   render_view_host->CopyFromBackingStore(
       gfx::Rect(),
       view->GetViewBounds().size(),
       base::Bind(&CaptureWebContentsFunction::CopyFromBackingStoreComplete,
-                 this));
+                 this),
+      preferred_format);

[piman, 2014/03/10 18:27:20]

The receiving code only accepts 8888.

[sivag, 2014/03/11 14:41:50]

Done.

   return true;
 }
 
 void CaptureWebContentsFunction::CopyFromBackingStoreComplete(
     bool succeeded,
     const SkBitmap& bitmap) {
   if (succeeded) {
     OnCaptureSuccess(bitmap);
     return;
   }
@@ skipping 59 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