Add ToString method for easy DLOG and debug support

chrome/common/custom_handlers/protocol_handler.cc

Index: chrome/common/custom_handlers/protocol_handler.cc
diff --git a/chrome/common/custom_handlers/protocol_handler.cc b/chrome/common/custom_handlers/protocol_handler.cc
index 4d61a415c62a3b1650c7044844f3fde115371492..0b77ad161334639879eb4631540dc66b42990dc1 100644
--- a/chrome/common/custom_handlers/protocol_handler.cc
+++ b/chrome/common/custom_handlers/protocol_handler.cc
@@ -70,6 +70,14 @@ DictionaryValue* ProtocolHandler::Encode() const {
   return d;
 }
 
+std::string ProtocolHandler::ToString() const {
+  return std::string("{ ") +

[Peter Kasting, 2012/06/22 22:10:06]

Nit: I'd probably write/wrap as:

  return "{ protocol=" + protocol_ + ", url=" + url_ + ", title=" + title_ +
      " }";

The more important bits here are, include the trailing underscores on your
variable names, and you can omit the std::string() at the beginning + not place
two spaces in a row if you combine the two opening string constants.  :)

[Steve McKay, 2012/06/22 22:40:01]

Sorry. Mailed before I compiled :(

Had to add some more code to make this compile (since not all fields are
strings). Doesn't really hold up now in the long "less wrapped" form. Happy to
rewrap differently if specify what you want.

+      " protocol=" + protocol +
+      ", url=" + url +
+      ", title=" + title +
+      " }";
+}
+
 bool ProtocolHandler::operator==(const ProtocolHandler& other) const {
   return protocol_ == other.protocol_ &&
     url_ == other.url_ &&