Clean up dependencies for disk_cache histogram_macros.h.

chrome/browser/net/http_pipelining_compatibility_client.cc

Index: chrome/browser/net/http_pipelining_compatibility_client.cc
diff --git a/chrome/browser/net/http_pipelining_compatibility_client.cc b/chrome/browser/net/http_pipelining_compatibility_client.cc
index 4ccafbbdb2875f8d05b3c57c04fa8ce065fcc7f8..8c8977693f68b4c47e8f1d92aece05485dd8d901 100644
--- a/chrome/browser/net/http_pipelining_compatibility_client.cc
+++ b/chrome/browser/net/http_pipelining_compatibility_client.cc
@@ -15,7 +15,6 @@
 #include "net/base/load_flags.h"
 #include "net/base/network_change_notifier.h"
 #include "net/base/request_priority.h"
-#include "net/disk_cache/histogram_macros.h"
 #include "net/http/http_network_layer.h"
 #include "net/http/http_network_session.h"
 #include "net/http/http_response_headers.h"
@@ -25,6 +24,12 @@
 #include "net/url_request/url_request_context.h"
 #include "net/url_request/url_request_context_getter.h"
 
+// This is included because the CACHE_HISTOGRAM_* macros allow dynamic metric
+// names.
+// TODO(gavinp): Clean up this dependency by moving the needed functionality
+// into base/.
+#include "net/disk_cache/histogram_macros.h"
+
 namespace chrome_browser_net {
 
 static const int kCanaryRequestId = 999;
@@ -364,8 +369,6 @@ void HttpPipeliningCompatibilityClient::OnCanaryFinished(
 
 void HttpPipeliningCompatibilityClient::OnRequestFinished(
     int request_id, internal::PipelineTestRequest::Status status) {
-  // The CACHE_HISTOGRAM_* macros are used, because they allow dynamic metric
-  // names.
   CACHE_HISTOGRAM_ENUMERATION(GetMetricName(request_id, "Status"),
                               status,
                               internal::PipelineTestRequest::STATUS_MAX);