Make NetLog take in callbacks that return Values

net/http/http_request_headers.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.
 
 #include "net/http/http_request_headers.h"
 
 #include "base/logging.h"
 #include "base/stringprintf.h"
 #include "base/string_split.h"
 #include "base/string_util.h"
+#include "base/values.h"
 #include "net/http/http_util.h"
 
 namespace net {
 
 const char HttpRequestHeaders::kGetMethod[] = "GET";
 const char HttpRequestHeaders::kAcceptCharset[] = "Accept-Charset";
 const char HttpRequestHeaders::kAcceptEncoding[] = "Accept-Encoding";
 const char HttpRequestHeaders::kAcceptLanguage[] = "Accept-Language";
 const char HttpRequestHeaders::kAuthorization[] = "Authorization";
 const char HttpRequestHeaders::kCacheControl[] = "Cache-Control";
@@ skipping 154 matching lines @@
       base::StringAppendF(&output, "%s: %s\r\n",
                           it->key.c_str(), it->value.c_str());
     } else {
       base::StringAppendF(&output, "%s:\r\n", it->key.c_str());
     }
   }
   output.append("\r\n");
   return output;
 }
 
+Value* HttpRequestHeaders::NetLogCallback(
+    const std::string* request_line,
+    NetLog::LogLevel /* log_level */) const {
+  DictionaryValue* dict = new DictionaryValue();
+  dict->SetString("line", *request_line);
+  ListValue* headers = new ListValue();
+  for (HeaderVector::const_iterator it = headers_.begin();
+       it != headers_.end(); ++it) {
+    headers->Append(
+        new StringValue(base::StringPrintf("%s: %s",
+                                           it->key.c_str(),
+                                           it->value.c_str())));
+  }
+  dict->Set("headers", headers);
+  return dict;
+}
+
+bool HttpRequestHeaders::FromNetLogParam(const base::Value* event_param,
+                                         HttpRequestHeaders* headers,
+                                         std::string* request_line) {
+  headers->Clear();
+  *request_line = "";
+
+  const base::DictionaryValue *dict;

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

nit: asterisk on left

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

Done.

+  // Not const because there's no overload of Dictionary::GetList that takes
+  // a const list.
+  base::ListValue *header_list;

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

nit: asterisk on left

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

Done.

+
+  if (!event_param ||
+      !event_param->GetAsDictionary(&dict) ||
+      !dict->GetList("headers", &header_list) ||
+      !dict->GetString("line", request_line)) {
+    return false;
+  }
+
+  for (base::ListValue::const_iterator it = header_list->begin();
+       it != header_list->end();
+       ++it) {
+    std::string header_line;
+    if (!(*it)->GetAsString(&header_line)) {
+      headers->Clear();
+      *request_line = "";
+      return false;
+    }
+    headers->AddHeaderFromString(header_line);
+  }
+  return true;
+}
+
 HttpRequestHeaders::HeaderVector::iterator
 HttpRequestHeaders::FindHeader(const base::StringPiece& key) {
   for (HeaderVector::iterator it = headers_.begin();
        it != headers_.end(); ++it) {
     if (key.length() == it->key.length() &&
         !base::strncasecmp(key.data(), it->key.data(), key.length()))
       return it;
   }
 
   return headers_.end();
 }
 
 HttpRequestHeaders::HeaderVector::const_iterator
 HttpRequestHeaders::FindHeader(const base::StringPiece& key) const {
   for (HeaderVector::const_iterator it = headers_.begin();
        it != headers_.end(); ++it) {
     if (key.length() == it->key.length() &&
         !base::strncasecmp(key.data(), it->key.data(), key.length()))
       return it;
   }
 
   return headers_.end();
 }
 
 }  // namespace net