Move malloc/partition_alloc memory usage functions to base

base/process/process_metrics.h

 // Copyright (c) 2013 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.
 
 // This file contains routines for gathering resource statistics for processes
 // running on the system.
 
 #ifndef BASE_PROCESS_PROCESS_METRICS_H_
 #define BASE_PROCESS_PROCESS_METRICS_H_
 
@@ skipping 525 matching lines @@
 // memory region.
 // Returns info on the first memory region at or after |address|, including
 // resident memory and share mode.
 // |size| and |info| are output parameters, only valid on Success.
 BASE_EXPORT MachVMRegionResult GetBasicInfo(mach_port_t task,
                                             mach_vm_size_t* size,
                                             mach_vm_address_t* address,
                                             vm_region_basic_info_64* info);
 #endif  // defined(OS_MACOSX) && !defined(OS_IOS)
 
+// Returns total memory usage of malloc.
+BASE_EXPORT size_t GetMallocUsage();

[Primiano Tucci (use gerrit), 2017/06/14 12:10:32]

I defer this to mark@ but just pointing out that  the other functions here seem
to be member functions. I'd probably make them such just for consistency even if
I realize that there is no functional requirement.

I think this all comes from the fact that one can get a  ProcessMetrics for
another process. But then if you look at the actual code, most of these
functions don't even look at process_; and always assume that proces_ == local.

[keishi, 2017/06/14 13:21:39]

Done.

+
+// Returns total memory usage of partition alloc.
+BASE_EXPORT size_t GetPartitionAllocUsage();
+
 }  // namespace base
 
 #endif  // BASE_PROCESS_PROCESS_METRICS_H_