Report Memory.Experimental.Renderer.HighestPrivateMemoryFootprint.* metric on Win, Mac and Linux.

Change-Id: Ia240c27b892d29530fc10d7457f7dae40016ba97
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2011624
Commit-Queue: Takashi Sakamoto <tasak@google.com>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Erik Chen <erikchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#734382}

third_party/blink/renderer/controller/BUILD.gn

@@ -56,8 +56,6 @@ jumbo_component("controller") {
     sources += [
       "crash_memory_metrics_reporter_impl.cc",
       "crash_memory_metrics_reporter_impl.h",
-      "highest_pmf_reporter.cc",
-      "highest_pmf_reporter.h",
       "memory_usage_monitor_posix.cc",
       "memory_usage_monitor_posix.h",
       "oom_intervention_impl.cc",
@@ -82,6 +80,14 @@ jumbo_component("controller") {
       "Foundation.framework",
     ]
   }
+  # HighestPmfReporter depends on MemoryUsageMonitor and MemoryUsageMonitor
+  # depends on platform specific code. Explicitly specify supported platforms.
+  if (is_linux || is_win || is_android || is_mac) {
+    sources += [
+      "highest_pmf_reporter.cc",
+      "highest_pmf_reporter.h",
+    ]
+  }
 
   configs -= [ "//build/config/compiler:default_symbols" ]
   configs += blink_symbols_config
@@ -173,16 +179,15 @@ jumbo_source_set("blink_unittests_sources") {
   }
   if (is_android) {
     sources += [
-      "highest_pmf_reporter_test.cc",
       "memory_usage_monitor_posix_test.cc",
-      "memory_usage_monitor_test.cc",
       "oom_intervention_impl_test.cc",
       "user_level_memory_pressure_signal_generator_test.cc",
     ]
   }
 
-  if (is_mac) {
+  if (is_linux || is_android || is_mac || is_win) {
     sources += [
+      "highest_pmf_reporter_test.cc",
       "memory_usage_monitor_test.cc",
     ]
   }

third_party/blink/renderer/controller/blink_initializer.cc

@@ -62,7 +62,6 @@
 
 #if defined(OS_ANDROID)
 #include "third_party/blink/renderer/controller/crash_memory_metrics_reporter_impl.h"
-#include "third_party/blink/renderer/controller/highest_pmf_reporter.h"
 #include "third_party/blink/renderer/controller/oom_intervention_impl.h"
 #include "third_party/blink/renderer/controller/user_level_memory_pressure_signal_generator.h"
 #endif
@@ -71,6 +70,11 @@
 #include "third_party/blink/renderer/controller/memory_usage_monitor_posix.h"
 #endif
 
+#if defined(OS_LINUX) || defined(OS_ANDROID) || defined(OS_MACOSX) || \
+    defined(OS_WIN)
+#include "third_party/blink/renderer/controller/highest_pmf_reporter.h"
+#endif
+
 namespace blink {
 
 namespace {
@@ -143,7 +147,10 @@ void InitializeCommon(Platform* platform, mojo::BinderMap* binders) {
   // Initialize CrashMemoryMetricsReporterImpl in order to assure that memory
   // allocation does not happen in OnOOMCallback.
   CrashMemoryMetricsReporterImpl::Instance();
+#endif
 
+#if defined(OS_LINUX) || defined(OS_ANDROID) || defined(OS_MACOSX) || \
+    defined(OS_WIN)
   // Start reporting the highest private memory footprint after the first
   // navigation.
   HighestPmfReporter::Instance();