Improve IPC logging for ChannelHandle and NPVariant_Param

content/common/webkit_param_traits.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // NOTE: New trait definitions that will be used by Chrome Frame must be placed
 // in common_param_traits2.cc.
 
 #include "content/common/webkit_param_traits.h"
 
 #include "base/string_number_conversions.h"
@@ skipping 284 matching lines @@
              (r->type == NPVARIANT_PARAM_NULL)) {
     result = true;
   } else {
     NOTREACHED();
   }
 
   return result;
 }
 
 void ParamTraits<NPVariant_Param>::Log(const param_type& p, std::string* l) {
+  l->append(StringPrintf("NPVariant_Param(%d, ", static_cast<int>(p.type)));
   if (p.type == NPVARIANT_PARAM_BOOL) {
     LogParam(p.bool_value, l);
   } else if (p.type == NPVARIANT_PARAM_INT) {
     LogParam(p.int_value, l);
   } else if (p.type == NPVARIANT_PARAM_DOUBLE) {
     LogParam(p.double_value, l);
   } else if (p.type == NPVARIANT_PARAM_STRING) {
     LogParam(p.string_value, l);
   } else if (p.type == NPVARIANT_PARAM_SENDER_OBJECT_ROUTING_ID ||
              p.type == NPVARIANT_PARAM_RECEIVER_OBJECT_ROUTING_ID) {
     LogParam(p.npobject_routing_id, l);
+  } else {
+    l->append("<none>");
   }
+  l->append(")");
 }
 
 void ParamTraits<NPIdentifier_Param>::Write(Message* m, const param_type& p) {
   webkit_glue::SerializeNPIdentifier(p.identifier, m);
 }
 
 bool ParamTraits<NPIdentifier_Param>::Read(const Message* m,
                                            void** iter,
                                            param_type* r) {
   return webkit_glue::DeserializeNPIdentifier(*m, iter, &r->identifier);
@@ skipping 43 matching lines @@
       ReadParam(m, iter, &p->ssl_valid) &&
       ReadParam(m, iter, &p->preferred) &&
       ReadParam(m, iter, &p->blacklisted_by_user);
 }
 void ParamTraits<webkit_glue::PasswordForm>::Log(const param_type& p,
                                                  std::string* l) {
   l->append("<PasswordForm>");
 }
 
 }  // namespace IPC