Add ProcessMetrics::CreateCurrentProcessMetrics() method.

Code that creates a ProcessMetrics object can be confusing because
ProcessMetrics::CreateProcessMetrics() has a different signature on Mac.
The extra Mac parameter isn't required when creating a ProcessMetrics
object for the current process (which is a common thing to do).

To allow simplification of code that creates a ProcessMetrics object
for the current process, this CL introduces a new
ProcessMetrics::CreateCurrentProcessMetrics() method, which has the
same signature on all platforms.

BUG=553266

Review URL: https://codereview.chromium.org/1433603004

Cr-Commit-Position: refs/heads/master@{#359199}

1 files changed, 1 insertions, 13 deletions

diff --git a/base/trace_event/process_memory_totals_dump_provider.cc b/base/trace_event/process_memory_totals_dump_provider.cc
index c057deb..83015f8 100644
--- a/base/trace_event/process_memory_totals_dump_provider.cc
+++ b/base/trace_event/process_memory_totals_dump_provider.cc
@@ -25,17 +25,6 @@ namespace trace_event {
 // static
 uint64 ProcessMemoryTotalsDumpProvider::rss_bytes_for_testing = 0;
 
-namespace {
-
-ProcessMetrics* CreateProcessMetricsForCurrentProcess() {
-#if !defined(OS_MACOSX) || defined(OS_IOS)
-  return ProcessMetrics::CreateProcessMetrics(GetCurrentProcessHandle());
-#else
-  return ProcessMetrics::CreateProcessMetrics(GetCurrentProcessHandle(), NULL);
-#endif
-}
-}  // namespace
-
 // static
 ProcessMemoryTotalsDumpProvider*
 ProcessMemoryTotalsDumpProvider::GetInstance() {
@@ -45,8 +34,7 @@ ProcessMemoryTotalsDumpProvider::GetInstance() {
 }
 
 ProcessMemoryTotalsDumpProvider::ProcessMemoryTotalsDumpProvider()
-    : process_metrics_(CreateProcessMetricsForCurrentProcess()) {
-}
+    : process_metrics_(ProcessMetrics::CreateCurrentProcessMetrics()) {}
 
 ProcessMemoryTotalsDumpProvider::~ProcessMemoryTotalsDumpProvider() {
 }