Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(27)

Side by Side Diff: chrome/browser/metrics/chrome_metrics_service_client.cc

Issue 2735443003: Use StringPiece for CreateUploader/NetMetricsLogUploader params. (Closed)
Patch Set: Nit and Rebase Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/metrics/chrome_metrics_service_client.h" 5 #include "chrome/browser/metrics/chrome_metrics_service_client.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <set> 9 #include <set>
10 #include <utility> 10 #include <utility>
(...skipping 495 matching lines...) Expand 10 before | Expand all | Expand 10 after
506 // |FinishedReceivingProfilerData()| when the task completes. 506 // |FinishedReceivingProfilerData()| when the task completes.
507 metrics::TrackingSynchronizer::FetchProfilerDataAsynchronously( 507 metrics::TrackingSynchronizer::FetchProfilerDataAsynchronously(
508 weak_ptr_factory_.GetWeakPtr()); 508 weak_ptr_factory_.GetWeakPtr());
509 } else { 509 } else {
510 CollectFinalHistograms(); 510 CollectFinalHistograms();
511 } 511 }
512 } 512 }
513 513
514 std::unique_ptr<metrics::MetricsLogUploader> 514 std::unique_ptr<metrics::MetricsLogUploader>
515 ChromeMetricsServiceClient::CreateUploader( 515 ChromeMetricsServiceClient::CreateUploader(
516 const std::string& server_url, 516 base::StringPiece server_url,
517 const std::string& mime_type, 517 base::StringPiece mime_type,
518 metrics::MetricsLogUploader::MetricServiceType service_type, 518 metrics::MetricsLogUploader::MetricServiceType service_type,
519 const base::Callback<void(int)>& on_upload_complete) { 519 const base::Callback<void(int)>& on_upload_complete) {
520 return std::unique_ptr<metrics::MetricsLogUploader>( 520 return std::unique_ptr<metrics::MetricsLogUploader>(
521 new metrics::NetMetricsLogUploader( 521 new metrics::NetMetricsLogUploader(
522 g_browser_process->system_request_context(), server_url, mime_type, 522 g_browser_process->system_request_context(), server_url, mime_type,
523 service_type, on_upload_complete)); 523 service_type, on_upload_complete));
524 } 524 }
525 525
526 base::TimeDelta ChromeMetricsServiceClient::GetStandardUploadInterval() { 526 base::TimeDelta ChromeMetricsServiceClient::GetStandardUploadInterval() {
527 return metrics::GetUploadInterval(); 527 return metrics::GetUploadInterval();
(...skipping 432 matching lines...) Expand 10 before | Expand all | Expand 10 after
960 ukm_service_->Purge(); 960 ukm_service_->Purge();
961 ukm_service_->ResetClientId(); 961 ukm_service_->ResetClientId();
962 } 962 }
963 // Signal service manager to enable/disable UKM based on new state. 963 // Signal service manager to enable/disable UKM based on new state.
964 UpdateRunningServices(); 964 UpdateRunningServices();
965 } 965 }
966 966
967 bool ChromeMetricsServiceClient::IsHistorySyncEnabledOnAllProfiles() { 967 bool ChromeMetricsServiceClient::IsHistorySyncEnabledOnAllProfiles() {
968 return SyncDisableObserver::IsHistorySyncEnabledOnAllProfiles(); 968 return SyncDisableObserver::IsHistorySyncEnabledOnAllProfiles();
969 } 969 }
OLDNEW
« no previous file with comments | « chrome/browser/metrics/chrome_metrics_service_client.h ('k') | chromecast/browser/metrics/cast_metrics_service_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698