RC: Bump default render CPU sampling interval to 10 minutes.

R=chrisha@chromium.org

Bug: 755840
Change-Id: Ife8febb03b0fb8d48b864b9987488d90893c4bbf
Reviewed-on: https://chromium-review.googlesource.com/1005408
Commit-Queue: Chris Hamilton <chrisha@chromium.org>
Reviewed-by: Chris Hamilton <chrisha@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551458}

chrome/browser/resource_coordinator/resource_coordinator_render_process_probe.cc

@@ -24,7 +24,8 @@ namespace resource_coordinator {
 
 namespace {
 
-const int kDefaultMeasurementIntervalInSeconds = 1;
+constexpr base::TimeDelta kDefaultMeasurementInterval =
+    base::TimeDelta::FromMinutes(10);
 
 base::LazyInstance<ResourceCoordinatorRenderProcessProbe>::DestructorAtExit
     g_probe = LAZY_INSTANCE_INITIALIZER;
@@ -71,8 +72,7 @@ class ResourceCoordinatorRenderProcessMetricsHandler
 ResourceCoordinatorRenderProcessProbe::ResourceCoordinatorRenderProcessProbe()
     : metrics_handler_(
           std::make_unique<ResourceCoordinatorRenderProcessMetricsHandler>()),
-      interval_ms_(
-          base::TimeDelta::FromSeconds(kDefaultMeasurementIntervalInSeconds)) {
+      interval_(kDefaultMeasurementInterval) {
   UpdateWithFieldTrialParams();
 }
 
@@ -178,7 +178,7 @@ void ResourceCoordinatorRenderProcessProbe::
     HandleRenderProcessMetricsOnUIThread() {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   if (metrics_handler_->HandleMetrics(render_process_info_map_)) {
-    timer_.Start(FROM_HERE, interval_ms_, this,
+    timer_.Start(FROM_HERE, interval_, this,
                  &ResourceCoordinatorRenderProcessProbe::
                      RegisterAliveRenderProcessesOnUIThread);
   }
@@ -200,7 +200,7 @@ void ResourceCoordinatorRenderProcessProbe::UpdateWithFieldTrialParams() {
   int64_t interval_ms = GetGRCRenderProcessCPUProfilingIntervalInMs();
 
   if (interval_ms > 0) {
-    interval_ms_ = base::TimeDelta::FromMilliseconds(interval_ms);
+    interval_ = base::TimeDelta::FromMilliseconds(interval_ms);
   }
 }
 

chrome/browser/resource_coordinator/resource_coordinator_render_process_probe.h

@@ -85,8 +85,8 @@ class ResourceCoordinatorRenderProcessProbe {
   // A map of currently running ProcessHandles to Process.
   RenderProcessInfoMap render_process_info_map_;
 
-  // Time duration between measurements in milliseconds.
-  base::TimeDelta interval_ms_;
+  // Time duration between measurements.
+  base::TimeDelta interval_;
 
   // Timer to signal the |ResourceCoordinatorRenderProcessProbe| instance
   // to conduct its measurements as a regular interval;