Coverity: Initialize member variables.

ppapi/native_client/src/shared/ppapi_proxy/ppp_instance_combined.cc

Index: ppapi/native_client/src/shared/ppapi_proxy/ppp_instance_combined.cc
diff --git a/ppapi/native_client/src/shared/ppapi_proxy/ppp_instance_combined.cc b/ppapi/native_client/src/shared/ppapi_proxy/ppp_instance_combined.cc
index b3af5904e0b3cf29cedf77fbc317e9e465eb8d41..b3aad7e15e04201ea9d89eb01d4a94b42f30b5f2 100644
--- a/ppapi/native_client/src/shared/ppapi_proxy/ppp_instance_combined.cc
+++ b/ppapi/native_client/src/shared/ppapi_proxy/ppp_instance_combined.cc
@@ -5,12 +5,14 @@
 #include "ppapi/native_client/src/shared/ppapi_proxy/ppp_instance_combined.h"
 
 #include <stdlib.h>
+#include <string.h>
 
 namespace ppapi_proxy {
 
 PPP_Instance_Combined::PPP_Instance_Combined()
     : initialized_(false),
       did_change_view_1_0_(NULL) {
+  memset(&instance_1_1_, 0, sizeof(instance_1_1_));

[dmichael (off chromium), 2012/02/06 18:38:07]

doesn't it suffice to just put instance_1_1_() in the initializer list?

[James Hawkins, 2012/02/07 21:09:21]

The default constructor doesn't initialize the POD types (in this case, the
function pointers).

[dmichael (off chromium), 2012/02/07 21:30:19]

instance_1_1_ is a POD struct. Adding instance_1_1_() to the initializer list
"value-initializes" instance_1_1_, which means the pointers are all set to NULL.
See, e.g.:
http://stackoverflow.com/questions/3803153/what-are-primitive-types-default-i...

That said, not everyone knows that, so memset may be clearer. I'm fine either
way.

 }
 
 void PPP_Instance_Combined::Init1_0(const PPP_Instance_1_0* instance_if) {