Pepper: Initialize PP_ImageDataDesc when creating ImageData.

Pepper: Initialize PP_ImageDataDesc when creating ImageData.
Clears memory so we don't send unitialized memory to the
renderer.

BUG=391023
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282222

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

Why are you working??

lgtm

https://codereview.chromium.org/383543004/diff/20001/ppapi/proxy/ppb_image_da...
File ppapi/proxy/ppb_image_data_proxy.cc (right):

https://codereview.chromium.org/383543004/diff/20001/ppapi/proxy/ppb_image_da...
ppapi/proxy/ppb_image_data_proxy.cc:500: memset(&desc, 0, sizeof(desc));
I think it would be enough to say
PP_ImageDataDesc desc();
or
PP_ImageDataDesc desc = {};
?

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

https://codereview.chromium.org/383543004/diff/20001/ppapi/proxy/ppb_image_da...
File ppapi/proxy/ppb_image_data_proxy.cc (right):

https://codereview.chromium.org/383543004/diff/20001/ppapi/proxy/ppb_image_da...
ppapi/proxy/ppb_image_data_proxy.cc:500: memset(&desc, 0, sizeof(desc));
On 2014/07/09 23:11:16, dmichael wrote:
> I think it would be enough to say
> PP_ImageDataDesc desc();
> or
> PP_ImageDataDesc desc = {};
> ?

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

[bbudge, 2014-07-09 23:35:06]

The CQ bit was checked by bbudge@chromium.org

[commit-bot: I haz the power, 2014-07-09 23:37:07]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/bbudge@chromium.org/383543004/40001

[commit-bot: I haz the power, 2014-07-10 02:54:16]

Change committed as 282222