Implement UMA log throttling for cellular connections

components/metrics/profiler/profiler_metrics_provider.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/metrics/profiler/profiler_metrics_provider.h"
 
 #include <ctype.h>
 #include <stddef.h>
 #include <string>
 #include <vector>
@@ skipping 58 matching lines @@
     tracked_object->set_process_id(process_id);
   }
 }
 
 }  // namespace
 
 ProfilerMetricsProvider::ProfilerMetricsProvider() {
 }
 
 ProfilerMetricsProvider::ProfilerMetricsProvider(
-    const base::Callback<bool(void)>& cellular_callback)
-    : cellular_callback_(cellular_callback) {
-}
+    const base::Callback<void(bool*)>& cellular_callback)
+    : cellular_callback_(cellular_callback) {}
 
 ProfilerMetricsProvider::~ProfilerMetricsProvider() {
 }
 
 void ProfilerMetricsProvider::ProvideGeneralMetrics(
     ChromeUserMetricsExtension* uma_proto) {
   DCHECK_EQ(0, uma_proto->profiler_event_size());
 
   for (auto& event : profiler_events_cache_) {
     uma_proto->add_profiler_event()->Swap(&event.second);
@@ skipping 27 matching lines @@
     for (const auto& event : past_events) {
       profiler_event->add_past_session_event(event);
     }
   }
 
   WriteProfilerData(process_data_phase, process_id, process_type,
                     profiler_event);
 }
 
 bool ProfilerMetricsProvider::IsCellularLogicEnabled() {
-  if (cellular_callback_.is_null())
-    return false;
-
-  return cellular_callback_.Run();
+  bool is_cellular_enabled = false;
+  if (!cellular_callback_.is_null()) {

[Alexei Svitkine (slow), 2016/03/29 16:29:11]

Nit: No {}'s for 1-liner ifs.

[gayane -on leave until 09-2017, 2016/03/31 01:38:24]

Done.

+    cellular_callback_.Run(&is_cellular_enabled);
+  }
+  return is_cellular_enabled;
 }
 
 }  // namespace metrics