Pepper: Make sure 'out' params are initialized for all code paths.

ppapi/proxy/ppb_instance_proxy.cc

Index: ppapi/proxy/ppb_instance_proxy.cc
diff --git a/ppapi/proxy/ppb_instance_proxy.cc b/ppapi/proxy/ppb_instance_proxy.cc
index 2780022d605b2e13f7efa82dbb956e71ff575379..f227e34968aaacf34538b17848411149a776f8b9 100644
--- a/ppapi/proxy/ppb_instance_proxy.cc
+++ b/ppapi/proxy/ppb_instance_proxy.cc
@@ -497,6 +497,9 @@ void PPB_Instance_Proxy::ZoomLimitsChanged(PP_Instance instance,
 
 PP_Var PPB_Instance_Proxy::GetDocumentURL(PP_Instance instance,
                                           PP_URLComponents_Dev* components) {
+  // Clear |components| so we don't send unitialized memory to the plugin.
+  // https://crbug.com/391023.
+  memset(components, 0, sizeof(*components));

[dmichael (off chromium), 2014/07/11 15:22:26]

ditto

[bbudge, 2014/07/11 17:01:55]

Backed out my change here. |components| may be NULL. If not, code below makes
sure components isn't uninitialized.

   ReceiveSerializedVarReturnValue result;
   PP_URLComponents_Dev url_components = {{0}};
   dispatcher()->Send(new PpapiHostMsg_PPBInstance_GetDocumentURL(