Add ToString method for easy DLOG and debug support

chrome/common/custom_handlers/protocol_handler.h

 // 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.
 
 #ifndef CHROME_COMMON_CUSTOM_HANDLERS_PROTOCOL_HANDLER_H_
 #define CHROME_COMMON_CUSTOM_HANDLERS_PROTOCOL_HANDLER_H_
 #pragma once
 
 #include <string>
 
@@ skipping 50 matching lines @@
   ProtocolHandler(const std::string& protocol,
                   const GURL& url,
                   const string16& title);
   ProtocolHandler();
 
   std::string protocol_;
   GURL url_;
   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_