Clean up dependencies for disk_cache histogram_macros.h.

chrome/browser/net/http_pipelining_compatibility_client.cc

 // Copyright (c) 2012 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 "chrome/browser/net/http_pipelining_compatibility_client.h"
 
 #include "base/metrics/field_trial.h"
 #include "base/metrics/histogram.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_split.h"
@@ skipping 348 matching lines @@
     StartTestRequests();
   } else {
     finished_callback_.Run(0);
   }
 }
 
 void HttpPipeliningCompatibilityClient::OnRequestFinished(
     int request_id, internal::PipelineTestRequest::Status status) {
   // The CACHE_HISTOGRAM_* macros are used, because they allow dynamic metric
   // names.
+  // TODO(gavinp): Clean up this dependency by moving the needed functionality
+  // into base/.
   CACHE_HISTOGRAM_ENUMERATION(GetMetricName(request_id, "Status"),
                               status,
                               internal::PipelineTestRequest::STATUS_MAX);
 
   ++num_finished_;
   if (status == internal::PipelineTestRequest::STATUS_SUCCESS) {
     ++num_succeeded_;
   }
   if (num_finished_ == requests_.size()) {
     UMA_HISTOGRAM_BOOLEAN("NetConnectivity.Pipeline.Success",
@@ skipping 151 matching lines @@
 
   content::BrowserThread::PostTask(
       content::BrowserThread::IO,
       FROM_HERE,
       base::Bind(&CollectPipeliningCapabilityStatsOnIOThread,
                  pipeline_test_server,
                  io_thread));
 }
 
 }  // namespace chrome_browser_net