Add ToString method for easy DLOG and debug support

chrome/common/custom_handlers/protocol_handler.h

Index: chrome/common/custom_handlers/protocol_handler.h
diff --git a/chrome/common/custom_handlers/protocol_handler.h b/chrome/common/custom_handlers/protocol_handler.h
index 8417ae8bfe05856a57cdae3132c26c4266ca34a4..7369391bbd85d78f550040d7e384daa96c23fcbf 100644
--- a/chrome/common/custom_handlers/protocol_handler.h
+++ b/chrome/common/custom_handlers/protocol_handler.h
@@ -68,4 +68,6 @@ class ProtocolHandler {
   string16 title_;
 };
 
+std::ostream& operator<<(std::ostream& os, const ProtocolHandler& handler);

[Peter Kasting, 2012/06/22 20:20:32]

Technically to use this you need to #include <ostream>.

This in turn pulls in the stream headers into anyone who #includes this.  That's
not a great thing (and bordering on a style guide violation, in that the style
guide tries to lock down streams fairly tightly).

The usual way we solve this is to instead provide a ToString() method or similar
that can return a string representation.  Then test code or other callers who
are already using streams can just do "<< obj.ToString()", and we don't taint
any more code than is necessary with streams.

[smckay, 2012/06/22 21:57:43]

Done.

Q: Does the general distaste for streams then apply to things like
std::stringstream?

+
 #endif  // CHROME_COMMON_CUSTOM_HANDLERS_PROTOCOL_HANDLER_H_