Make NetLog take in callbacks that return Values

net/http/http_request_headers.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.
 //
 // HttpRequestHeaders manages the request headers.
 // It maintains these in a vector of header key/value pairs, thereby maintaining
 // the order of the headers.  This means that any lookups are linear time
 // operations.
 
 #ifndef NET_HTTP_HTTP_REQUEST_HEADERS_H_
 #define NET_HTTP_HTTP_REQUEST_HEADERS_H_
 #pragma once
 
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/string_piece.h"
 #include "net/base/net_export.h"
+#include "net/base/net_log.h"
+
+namespace base {
+  class Value;

[eroman, 2012/06/06 21:27:11]

nit: de-indent. Isn't value already declared in net_log.h?

[mmenke, 2012/06/07 19:20:46]

Yea, it is.  Removed.

+}
 
 namespace net {
 
 class NET_EXPORT HttpRequestHeaders {
  public:
   struct HeaderKeyValuePair {
     HeaderKeyValuePair();
     HeaderKeyValuePair(const base::StringPiece& key,
                        const base::StringPiece& value);
 
@@ skipping 108 matching lines @@
 
   void Swap(HttpRequestHeaders* other) {
     headers_.swap(other->headers_);
   }
 
   // Serializes HttpRequestHeaders to a string representation.  Joins all the
   // header keys and values with ": ", and inserts "\r\n" between each header
   // line, and adds the trailing "\r\n".
   std::string ToString() const;
 
+  // Takes in the request line and returns a Value for use with the NetLog
+  // containing both the request line and all headers fields.
+  base::Value* NetLogCallback(const std::string* request_line,
+                              NetLog::LogLevel log_level) const;
+
+  // Takes in a Value created by the above function, and attempts to extract the
+  // request line and create a copy of the original headers.  Returns true on
+  // success.  On failure, clears |headers| and |request_line|.
+  // TODO(mmenke):  Long term, we want to remove this, and migrate external
+  //                consumers to be NetworkDelegates.
+  static bool FromNetLogParam(const base::Value* event_param,
+                              HttpRequestHeaders* headers,
+                              std::string* request_line);
+
  private:
   HeaderVector::iterator FindHeader(const base::StringPiece& key);
   HeaderVector::const_iterator FindHeader(const base::StringPiece& key) const;
 
   HeaderVector headers_;
 
   // Allow the copy construction and operator= to facilitate copying in
-  // HttpRequestInfo.
+  // HttpRequestHeaders.
   // TODO(willchan): Investigate to see if we can remove the need to copy
-  // HttpRequestInfo.
+  // HttpRequestHeaders.
   // DISALLOW_COPY_AND_ASSIGN(HttpRequestHeaders);
 };
 
 }  // namespace net
 
 #endif  // NET_HTTP_HTTP_REQUEST_HEADERS_H_