Fix array out of bounds access in loop iteration in gpu ipc log

gpu/ipc/gpu_command_buffer_traits.cc

Index: gpu/ipc/gpu_command_buffer_traits.cc
diff --git a/gpu/ipc/gpu_command_buffer_traits.cc b/gpu/ipc/gpu_command_buffer_traits.cc
index 8d2148de0f7842d9741cbb61ba6fed3f18450810..72bc6333fac08bb9b328f746f8ff9ee90e8da624 100644
--- a/gpu/ipc/gpu_command_buffer_traits.cc
+++ b/gpu/ipc/gpu_command_buffer_traits.cc
@@ -115,10 +115,10 @@ bool ParamTraits<gpu::ValueState> ::Read(const Message* m,
 
 void ParamTraits<gpu::ValueState>::Log(const param_type& p, std::string* l) {
   l->append("<ValueState (");
-  for (size_t i = 0; i < sizeof(p.int_value); ++i)
+  for (size_t i = 0; i < sizeof(p.int_value) / sizeof(p.int_value[0]); ++i)

[piman, 2015/10/06 20:04:45]

nit: Can you use arraysize here?
Or even better use range for:
for (int value : p.int_value)
  *l += base::StringPrintf("%i ", value);

[RaviKasibhatla, 2015/10/07 11:58:34]

Done.

     *l += base::StringPrintf("%i ", p.int_value[i]);
   l->append(" int values ");
-  for (size_t i = 0; i < sizeof(p.float_value); ++i)
+  for (size_t i = 0; i < sizeof(p.float_value) / sizeof(p.float_value[0]); ++i)

[piman, 2015/10/06 20:04:45]

nit: same here.

[RaviKasibhatla, 2015/10/07 11:58:33]

Done.

     *l += base::StringPrintf("%f ", p.float_value[i]);
   l->append(" float values)>");
 }