<webview>: Fix missing transparency in captureVisibleRegion().

extensions/browser/api/web_contents_capture_client.cc

Index: extensions/browser/api/web_contents_capture_client.cc
diff --git a/extensions/browser/api/web_contents_capture_client.cc b/extensions/browser/api/web_contents_capture_client.cc
index 78c3760667ec99357e349436c3d47990c00f47dd..6e2e5e221e00971998fee9871579d4c649c3e678 100644
--- a/extensions/browser/api/web_contents_capture_client.cc
+++ b/extensions/browser/api/web_contents_capture_client.cc
@@ -120,8 +120,9 @@ bool WebContentsCaptureClient::EncodeBitmap(const SkBitmap& bitmap,
     case api::extension_types::IMAGE_FORMAT_PNG:
       encoded =
           gfx::PNGCodec::EncodeBGRASkBitmap(bitmap,
-                                            true,  // Discard transparency.
-                                            &data);
+                                            /* Don't discard transparency if
+                                             * client allows transparency. */

[Finnur, 2016/04/07 10:08:33]

Please avoid multi-line comments. This comment doesn't really help with
anything, so I would just remove it.

[avallee, 2016/04/07 14:21:16]

Replaced with a variable that documents what's going on.

+                                            !ClientAllowsTransparency(), &data);
       mime_type = kMimeTypePng;
       break;
     default: