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

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

Issue 2735443003: Use StringPiece for CreateUploader/NetMetricsLogUploader params. (Closed)
Patch Set: 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 496 matching lines...) Expand 10 before | Expand all | Expand 10 after
507 // |FinishedReceivingProfilerData()| when the task completes. 507 // |FinishedReceivingProfilerData()| when the task completes.
508 metrics::TrackingSynchronizer::FetchProfilerDataAsynchronously( 508 metrics::TrackingSynchronizer::FetchProfilerDataAsynchronously(
509 weak_ptr_factory_.GetWeakPtr()); 509 weak_ptr_factory_.GetWeakPtr());
510 } else { 510 } else {
511 CollectFinalHistograms(); 511 CollectFinalHistograms();
512 } 512 }
513 } 513 }
514 514
515 std::unique_ptr<metrics::MetricsLogUploader> 515 std::unique_ptr<metrics::MetricsLogUploader>
516 ChromeMetricsServiceClient::CreateUploader( 516 ChromeMetricsServiceClient::CreateUploader(
517 const std::string& server_url, 517 const base::StringPiece& server_url,
518 const std::string& mime_type, 518 const base::StringPiece& mime_type,
519 metrics::MetricsLogUploader::MetricServiceType service_type, 519 metrics::MetricsLogUploader::MetricServiceType service_type,
520 const base::Callback<void(int)>& on_upload_complete) { 520 const base::Callback<void(int)>& on_upload_complete) {
521 return std::unique_ptr<metrics::MetricsLogUploader>( 521 return std::unique_ptr<metrics::MetricsLogUploader>(
522 new metrics::NetMetricsLogUploader( 522 new metrics::NetMetricsLogUploader(
523 g_browser_process->system_request_context(), server_url, mime_type, 523 g_browser_process->system_request_context(), server_url, mime_type,
524 service_type, on_upload_complete)); 524 service_type, on_upload_complete));
525 } 525 }
526 526
527 base::TimeDelta ChromeMetricsServiceClient::GetStandardUploadInterval() { 527 base::TimeDelta ChromeMetricsServiceClient::GetStandardUploadInterval() {
528 return metrics::GetUploadInterval(); 528 return metrics::GetUploadInterval();
(...skipping 433 matching lines...) Expand 10 before | Expand all | Expand 10 after
962 ukm_service_->Purge(); 962 ukm_service_->Purge();
963 ukm_service_->ResetClientId(); 963 ukm_service_->ResetClientId();
964 } 964 }
965 // Signal service manager to enable/disable UKM based on new state. 965 // Signal service manager to enable/disable UKM based on new state.
966 UpdateRunningServices(); 966 UpdateRunningServices();
967 } 967 }
968 968
969 bool ChromeMetricsServiceClient::IsHistorySyncEnabledOnAllProfiles() { 969 bool ChromeMetricsServiceClient::IsHistorySyncEnabledOnAllProfiles() {
970 return SyncDisableObserver::IsHistorySyncEnabledOnAllProfiles(); 970 return SyncDisableObserver::IsHistorySyncEnabledOnAllProfiles();
971 } 971 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698