Use StringPiece for CreateUploader/NetMetricsLogUploader params.

components/metrics/test_metrics_log_uploader.h

Index: components/metrics/test_metrics_log_uploader.h
diff --git a/components/metrics/test_metrics_log_uploader.h b/components/metrics/test_metrics_log_uploader.h
index 196370a4198993e969d6b835647e316bab24c3c5..e887d2deb658ed4434411a95b21114fb9a7082f4 100644
--- a/components/metrics/test_metrics_log_uploader.h
+++ b/components/metrics/test_metrics_log_uploader.h
@@ -11,11 +11,8 @@ namespace metrics {
 
 class TestMetricsLogUploader : public MetricsLogUploader {
  public:
-  TestMetricsLogUploader(const std::string& server_url,
-                         const std::string& mime_type,
-                         MetricsLogUploader::MetricServiceType service_type,
-                         const base::Callback<void(int)>& on_upload_complete);
-  ~TestMetricsLogUploader() override;
+  TestMetricsLogUploader(const base::Callback<void(int)>& on_upload_complete);

[Alexei Svitkine (slow), 2017/03/06 21:07:46]

explicit

[Steven Holte, 2017/03/06 21:22:11]

Done.

+  ~TestMetricsLogUploader();
 
   // Mark the current upload complete with the given response code.
   void CompleteUpload(int response_code);
@@ -28,6 +25,7 @@ class TestMetricsLogUploader : public MetricsLogUploader {
   void UploadLog(const std::string& compressed_log_data,
                  const std::string& log_hash) override;
 
+  const base::Callback<void(int)> on_upload_complete_;
   bool is_uploading_;
 
   DISALLOW_COPY_AND_ASSIGN(TestMetricsLogUploader);