Pepper: Initialize PP_ImageDataDesc when creating ImageData.

ppapi/proxy/ppb_image_data_proxy.cc

Index: ppapi/proxy/ppb_image_data_proxy.cc
diff --git a/ppapi/proxy/ppb_image_data_proxy.cc b/ppapi/proxy/ppb_image_data_proxy.cc
index 87b49a777a1caf669be091172a24d52acfdecf0a..d2d7555563542071b87d0a9412077704e4b87982 100644
--- a/ppapi/proxy/ppb_image_data_proxy.cc
+++ b/ppapi/proxy/ppb_image_data_proxy.cc
@@ -495,6 +495,9 @@ PP_Resource PPB_ImageData_Proxy::CreateProxyResource(
 
   HostResource result;
   PP_ImageDataDesc desc;
+  // Don't send an uninitialized |desc| to the renderer.
+  // https://crbug.com/391023.
+  memset(&desc, 0, sizeof(desc));

[dmichael (off chromium), 2014/07/09 23:11:16]

I think it would be enough to say
PP_ImageDataDesc desc();
or
PP_ImageDataDesc desc = {};
?

[bbudge, 2014/07/09 23:34:53]

Better. I'll use the latter which is clearer (and compiles!)

   switch (type) {
     case PPB_ImageData_Shared::SIMPLE: {
       ppapi::proxy::SerializedHandle image_handle_wrapper;