Reenable Purge+Suspend Memory Growth metrics.

content/renderer/render_thread_impl.cc

 // Copyright (c) 2012 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 "content/renderer/render_thread_impl.h"
 
 #include <algorithm>
 #include <limits>
 #include <map>
 #include <utility>
 #include <vector>
 
 #include "base/allocator/allocator_extension.h"
 #include "base/at_exit.h"
 #include "base/command_line.h"
 #include "base/debug/crash_logging.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/memory/discardable_memory_allocator.h"
 #include "base/memory/memory_coordinator_client_registry.h"
 #include "base/memory/ptr_util.h"
 #include "base/memory/shared_memory.h"
 #include "base/message_loop/message_loop.h"
 #include "base/metrics/field_trial.h"
+#include "base/metrics/histogram_functions.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/path_service.h"
 #include "base/process/process_metrics.h"
 #include "base/run_loop.h"
 #include "base/strings/string16.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_split.h"
 #include "base/strings/sys_string_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/threading/sequenced_worker_pool.h"
@@ skipping 1652 matching lines @@
     return;
 
   base::MemoryCoordinatorClientRegistry::GetInstance()->PurgeMemory();
   needs_to_record_first_active_paint_ = true;
 
   RendererMemoryMetrics memory_metrics;
   if (!GetRendererMemoryMetrics(&memory_metrics))
     return;
 
   purge_and_suspend_memory_metrics_ = memory_metrics;
+  GetRendererScheduler()->DefaultTaskRunner()->PostDelayedTask(
+      FROM_HERE,
+      base::Bind(&RenderThreadImpl::RecordPurgeAndSuspendMemoryGrowthMetrics,
+                 base::Unretained(this), process_foregrounded_count_,
+                 RECORD_30MIN_AFTER_PURGE),
+      base::TimeDelta::FromMinutes(30));
+  GetRendererScheduler()->DefaultTaskRunner()->PostDelayedTask(
+      FROM_HERE,
+      base::Bind(&RenderThreadImpl::RecordPurgeAndSuspendMemoryGrowthMetrics,
+                 base::Unretained(this), process_foregrounded_count_,
+                 RECORD_60MIN_AFTER_PURGE),
+      base::TimeDelta::FromMinutes(60));
+  GetRendererScheduler()->DefaultTaskRunner()->PostDelayedTask(
+      FROM_HERE,
+      base::Bind(&RenderThreadImpl::RecordPurgeAndSuspendMemoryGrowthMetrics,
+                 base::Unretained(this), process_foregrounded_count_,
+                 RECORD_90MIN_AFTER_PURGE),
+      base::TimeDelta::FromMinutes(90));
 }
 
 // TODO(tasak): Replace the following GetMallocUsage() with memory-infra
 // when it is possible to run memory-infra without tracing.
 #if defined(OS_WIN)
 namespace {
 
 static size_t GetMallocUsage() {
   // Iterate through whichever heap the CRT is using.
   HANDLE crt_heap = reinterpret_cast<HANDLE>(_get_heap_handle());
@@ skipping 81 matching lines @@
       total_allocated / render_view_count / 1024 / 1024;
 
   return true;
 }
 
 #define GET_MEMORY_GROWTH(current, previous, allocator) \
   (current.allocator > previous.allocator               \
        ? current.allocator - previous.allocator         \
        : 0)
 
-#define UMA_HISTOGRAM_MEMORY_GROWTH_KB(basename, suffix, memory_usage) \
-  {                                                                    \
-    std::string histogram_name =                                       \
-        base::StringPrintf("%s.%s", basename, suffix);                 \
-    UMA_HISTOGRAM_MEMORY_KB(histogram_name, memory_usage);             \
-  }
+#define UMA_PURGE_AND_SUSPEND_MEMORY_GROWTH(suffix, memory_metrics)        \

[Ilya Sherman, 2017/05/23 15:21:58]

Rather than using a macro here, please revert to passing a string suffix, and
just append the suffix below.

[tasak, 2017/05/24 05:08:19]

I see. I misunderstood the previous comment.
So by using base::UmaHistogramMemoryKB, it is possible to use generated
histogram names....

Done.

+  base::UmaHistogramMemoryKB(                                              \
+      "PurgeAndSuspend.Experimental.MemoryGrowth.PartitionAllocKB" suffix, \

[Ilya Sherman, 2017/05/23 15:21:58]

By the way, I think you lost the final dot in the histogram name?

[tasak, 2017/05/24 05:08:19]

Yeah, it's a critical bug... Thanks.
Fixed by reverting the code.

+      GET_MEMORY_GROWTH(memory_metrics, purge_and_suspend_memory_metrics_, \
+                        partition_alloc_kb));                              \
+  base::UmaHistogramMemoryKB(                                              \
+      "PurgeAndSuspend.Experimental.MemoryGrowth.BlinkGCKB" suffix,        \
+      GET_MEMORY_GROWTH(memory_metrics, purge_and_suspend_memory_metrics_, \
+                        blink_gc_kb));                                     \
+  base::UmaHistogramMemoryKB(                                              \
+      "PurgeAndSuspend.Experimental.MemoryGrowth.MallocKB" suffix,         \
+      GET_MEMORY_GROWTH(memory_metrics, purge_and_suspend_memory_metrics_, \
+                        malloc_mb) *                                       \
+          1024);                                                           \
+  base::UmaHistogramMemoryKB(                                              \
+      "PurgeAndSuspend.Experimental.MemoryGrowth.DiscardableKB" suffix,    \
+      GET_MEMORY_GROWTH(memory_metrics, purge_and_suspend_memory_metrics_, \
+                        discardable_kb));                                  \
+  base::UmaHistogramMemoryKB(                                              \
+      "PurgeAndSuspend.Experimental.MemoryGrowth."                         \
+      "V8MainThreadIsolateKB" suffix,                                      \
+      GET_MEMORY_GROWTH(memory_metrics, purge_and_suspend_memory_metrics_, \
+                        v8_main_thread_isolate_mb) *                       \
+          1024);                                                           \
+  base::UmaHistogramMemoryKB(                                              \
+      "PurgeAndSuspend.Experimental.MemoryGrowth.TotalAllocatedKB" suffix, \
+      GET_MEMORY_GROWTH(memory_metrics, purge_and_suspend_memory_metrics_, \
+                        total_allocated_mb) *                              \
+          1024);
 
 void RenderThreadImpl::RecordPurgeAndSuspendMemoryGrowthMetrics(
-    const char* suffix,
-    int foregrounded_count_when_purged) {
+    int foregrounded_count_when_purged,
+    RenderThreadImpl::RecordMemoryGrowthTiming record_timing) {
   // If this renderer is resumed, we should not update UMA.
   if (!RendererIsHidden())
     return;
   if (foregrounded_count_when_purged != process_foregrounded_count_)
     return;
 
   RendererMemoryMetrics memory_metrics;
   if (!GetRendererMemoryMetrics(&memory_metrics))
     return;
 
-  UMA_HISTOGRAM_MEMORY_GROWTH_KB(
-      "PurgeAndSuspend.Experimental.MemoryGrowth.PartitionAllocKB", suffix,
-      GET_MEMORY_GROWTH(memory_metrics, purge_and_suspend_memory_metrics_,
-                        partition_alloc_kb));
-  UMA_HISTOGRAM_MEMORY_GROWTH_KB(
-      "PurgeAndSuspend.Experimental.MemoryGrowth.BlinkGCKB", suffix,
-      GET_MEMORY_GROWTH(memory_metrics, purge_and_suspend_memory_metrics_,
-                        blink_gc_kb));
-  UMA_HISTOGRAM_MEMORY_GROWTH_KB(
-      "PurgeAndSuspend.Experimental.MemoryGrowth.MallocKB", suffix,
-      GET_MEMORY_GROWTH(memory_metrics, purge_and_suspend_memory_metrics_,
-                        malloc_mb) *
-          1024);
-  UMA_HISTOGRAM_MEMORY_GROWTH_KB(
-      "PurgeAndSuspend.Experimental.MemoryGrowth.DiscardableKB", suffix,
-      GET_MEMORY_GROWTH(memory_metrics, purge_and_suspend_memory_metrics_,
-                        discardable_kb));
-  UMA_HISTOGRAM_MEMORY_GROWTH_KB(
-      "PurgeAndSuspend.Experimental.MemoryGrowth.V8MainThreadIsolateKB", suffix,
-      GET_MEMORY_GROWTH(memory_metrics, purge_and_suspend_memory_metrics_,
-                        v8_main_thread_isolate_mb) *
-          1024);
-  UMA_HISTOGRAM_MEMORY_GROWTH_KB(
-      "PurgeAndSuspend.Experimental.MemoryGrowth.TotalAllocatedKB", suffix,
-      GET_MEMORY_GROWTH(memory_metrics, purge_and_suspend_memory_metrics_,
-                        total_allocated_mb) *
-          1024);
+  switch (record_timing) {
+    case RECORD_30MIN_AFTER_PURGE:
+      UMA_PURGE_AND_SUSPEND_MEMORY_GROWTH("30min", memory_metrics);
+      break;
+    case RECORD_60MIN_AFTER_PURGE:
+      UMA_PURGE_AND_SUSPEND_MEMORY_GROWTH("60min", memory_metrics);
+      break;
+    case RECORD_90MIN_AFTER_PURGE:
+      UMA_PURGE_AND_SUSPEND_MEMORY_GROWTH("90min", memory_metrics);
+      break;
+  }
 }
 
 scoped_refptr<gpu::GpuChannelHost> RenderThreadImpl::EstablishGpuChannelSync() {
   TRACE_EVENT0("gpu", "RenderThreadImpl::EstablishGpuChannelSync");
 
   if (gpu_channel_) {
     // Do nothing if we already have a GPU channel or are already
     // establishing one.
     if (!gpu_channel_->IsLost())
       return gpu_channel_;
@@ skipping 602 matching lines @@
   }
 }
 
 void RenderThreadImpl::OnRendererInterfaceRequest(
     mojom::RendererAssociatedRequest request) {
   DCHECK(!renderer_binding_.is_bound());
   renderer_binding_.Bind(std::move(request));
 }
 
 }  // namespace content