Index: content/browser/renderer_host/render_widget_host_view_android.cc |
diff --git a/content/browser/renderer_host/render_widget_host_view_android.cc b/content/browser/renderer_host/render_widget_host_view_android.cc |
index 66dcff23c33df2eecd21aec15ee49c7937d0a720..4f53cd0bf5291fbf61c8648ea3c666ceb0d03ab1 100644 |
--- a/content/browser/renderer_host/render_widget_host_view_android.cc |
+++ b/content/browser/renderer_host/render_widget_host_view_android.cc |
@@ -305,8 +305,7 @@ |
helper->ReadbackTextureSync(texture, |
gfx::Rect(bitmap.size()), |
- static_cast<unsigned char*> (bitmap.pixels()), |
- SkBitmap::kARGB_8888_Config); |
+ static_cast<unsigned char*> (bitmap.pixels())); |
blink::WebGraphicsContext3D* context = |
ImageTransportFactoryAndroid::GetInstance()->GetContext3D(); |
@@ -626,23 +625,12 @@ |
void RenderWidgetHostViewAndroid::CopyFromCompositingSurface( |
const gfx::Rect& src_subrect, |
const gfx::Size& dst_size, |
- const base::Callback<void(bool, const SkBitmap&)>& callback, |
- bool readback_config_rgb565) { |
+ const base::Callback<void(bool, const SkBitmap&)>& callback) { |
if (!using_synchronous_compositor_ && !IsSurfaceAvailableForCopy()) { |
callback.Run(false, SkBitmap()); |
return; |
} |
- ImageTransportFactoryAndroid* factory = |
- ImageTransportFactoryAndroid::GetInstance(); |
- GLHelper* gl_helper = factory->GetGLHelper(); |
- if (!gl_helper) |
- return; |
- bool check_rgb565_support = gl_helper->CanUseRgb565Readback(); |
- if (readback_config_rgb565 && !check_rgb565_support) { |
- LOG(ERROR) << "Readbackformat rgb565 not supported"; |
- callback.Run(false, SkBitmap()); |
- return; |
- } |
+ |
const gfx::Display& display = |
gfx::Screen::GetNativeScreen()->GetPrimaryDisplay(); |
float device_scale_factor = display.device_scale_factor(); |
@@ -658,6 +646,7 @@ |
SynchronousCopyContents(src_subrect_in_pixel, dst_size_in_pixel, callback); |
return; |
} |
+ |
scoped_ptr<cc::CopyOutputRequest> request; |
if (src_subrect_in_pixel.size() == dst_size_in_pixel) { |
request = cc::CopyOutputRequest::CreateBitmapRequest(base::Bind( |
@@ -668,7 +657,6 @@ |
request = cc::CopyOutputRequest::CreateRequest(base::Bind( |
&RenderWidgetHostViewAndroid::PrepareTextureCopyOutputResult, |
dst_size_in_pixel, |
- readback_config_rgb565, |
callback)); |
} |
request->set_area(src_subrect_in_pixel); |
@@ -1372,7 +1360,6 @@ |
// static |
void RenderWidgetHostViewAndroid::PrepareTextureCopyOutputResult( |
const gfx::Size& dst_size_in_pixel, |
- bool readback_config_rgb565, |
const base::Callback<void(bool, const SkBitmap&)>& callback, |
scoped_ptr<cc::CopyOutputResult> result) { |
DCHECK(result->HasTexture()); |
@@ -1383,12 +1370,8 @@ |
return; |
scoped_ptr<SkBitmap> bitmap(new SkBitmap); |
- SkBitmap::Config bitmap_config = readback_config_rgb565 ? |
- SkBitmap::kRGB_565_Config : |
- SkBitmap::kARGB_8888_Config; |
- bitmap->setConfig(bitmap_config, |
- dst_size_in_pixel.width(), |
- dst_size_in_pixel.height(), |
+ bitmap->setConfig(SkBitmap::kARGB_8888_Config, |
+ dst_size_in_pixel.width(), dst_size_in_pixel.height(), |
0, kOpaque_SkAlphaType); |
if (!bitmap->allocPixels()) |
return; |
@@ -1419,7 +1402,6 @@ |
gfx::Rect(result->size()), |
dst_size_in_pixel, |
pixels, |
- readback_config_rgb565, |
base::Bind(&CopyFromCompositingSurfaceFinished, |
callback, |
base::Passed(&release_callback), |