Chromium Code Reviews| Index: components/metrics/metrics_service.cc |
| diff --git a/components/metrics/metrics_service.cc b/components/metrics/metrics_service.cc |
| index 7e31ddb93156bf301ef333bd5cbe71bb7acc9691..071c96897103c1db2a9f4690cb6d6f6f616336c1 100644 |
| --- a/components/metrics/metrics_service.cc |
| +++ b/components/metrics/metrics_service.cc |
| @@ -232,21 +232,6 @@ void MarkAppCleanShutdownAndCommit(CleanExitBeacon* clean_exit_beacon, |
| } |
| #endif // defined(OS_ANDROID) || defined(OS_IOS) |
| -// Determines if current log should be sent based on sampling rate. Returns true |
| -// if the sampling rate is not set. |
| -bool ShouldUploadLog() { |
|
Alexei Svitkine (slow)
2016/05/13 21:11:56
Nit: Can you remove some the includes needed for t
gayane -on leave until 09-2017
2016/05/16 15:33:04
Done.
|
| - std::string probability_str = variations::GetVariationParamValue( |
| - "UMA_EnableCellularLogUpload", "Sample_Probability"); |
| - if (probability_str.empty()) |
| - return true; |
| - |
| - int probability; |
| - // In case specified sampling rate is invalid. |
| - if (!base::StringToInt(probability_str, &probability)) |
| - return true; |
| - return base::RandInt(1, 100) <= probability; |
| -} |
| - |
| } // namespace |
| // static |
| @@ -983,11 +968,6 @@ void MetricsService::SendStagedLog() { |
| DCHECK(!log_upload_in_progress_); |
| log_upload_in_progress_ = true; |
| - if (!ShouldUploadLog()) { |
| - SkipAndDiscardUpload(); |
| - return; |
| - } |
| - |
| if (!log_uploader_) { |
| log_uploader_ = client_->CreateUploader( |
| base::Bind(&MetricsService::OnLogUploadComplete, |