Chromium Code Reviews| Index: components/metrics/metrics_service.cc |
| diff --git a/components/metrics/metrics_service.cc b/components/metrics/metrics_service.cc |
| index 315229b11d1f0e73a6fcf399d4427d64aa47b748..2c962bc7526da9a3dcda94158d483acfee0bf123 100644 |
| --- a/components/metrics/metrics_service.cc |
| +++ b/components/metrics/metrics_service.cc |
| @@ -882,13 +882,19 @@ void MetricsService::SendNextLog() { |
| // Proceed to stage the log for upload if log size satisfies cellular log |
| // upload constrains. |
| - if (client_->IsUMACellularUploadLogicEnabled() && |
| + bool upload_canceled = false; |
| + bool is_cellular_logic = client_->IsUMACellularUploadLogicEnabled(); |
| + if (is_cellular_logic && |
| !data_use_tracker_->ShouldUploadLogOnCellular( |
| log_manager_.staged_log_hash().size())) { |
| scheduler_->UploadCancelled(); |
| + upload_canceled = true; |
| } else { |
| SendStagedLog(); |
| } |
| + if (is_cellular_logic) |
|
Alexei Svitkine (slow)
2016/04/11 16:04:49
Nit: {}
gayane -on leave until 09-2017
2016/04/11 17:42:25
Done.
|
| + UMA_HISTOGRAM_BOOLEAN("UMA.LogUpload.Canceled.CellularConstraint", |
| + upload_canceled); |
| } |
| bool MetricsService::ProvidersHaveInitialStabilityMetrics() { |