Report memory retained by memory allocator internals.

webkit/glue/webkitplatformsupport_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 "webkit/glue/webkitplatformsupport_impl.h"
 
 #if defined(OS_LINUX)
 #include <malloc.h>
 #endif
 
 #include <math.h>
 
 #include <vector>
 
+#include "base/allocator/allocator_extension.h"
 #include "base/bind.h"
 #include "base/debug/trace_event.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/singleton.h"
 #include "base/message_loop.h"
 #include "base/metrics/histogram.h"
 #include "base/metrics/stats_counters.h"
 #include "base/platform_file.h"
 #include "base/process_util.h"
 #include "base/rand_util.h"
@@ skipping 764 matching lines @@
   return base::SysInfo::DalvikHeapSizeMB() / 8;
 }
 #endif
 
 bool WebKitPlatformSupportImpl::processMemorySizesInBytes(
     size_t* private_bytes,
     size_t* shared_bytes) {
   return CurrentProcessMetrics()->GetMemoryBytes(private_bytes, shared_bytes);
 }
 
+bool WebKitPlatformSupportImpl::memoryAllocatorInternalsInBytes(size_t* size) {
+  size_t heap_size, allocated_bytes, unmapped_bytes;
+  if (base::allocator::GetProperty("generic.heap_size", &heap_size) &&

[darin (slow to review), 2012/10/18 18:02:54]

nit: Shouldn't there be some named constants for these strings?  It seems
fragile to hard-code these string literals here.

Alternatively, would it make sense for base::allocator:: to declare a method
that corresponds to memoryAllocatorInternalsInBytes?

One more note: I find the name memoryAllocatorInternalsInBytes to be quite
mysterious?  What does it mean?  Is it trying to count the number of bytes that
get used up by the allocator's own data structures?

[alph-g, 2012/10/22 12:19:41]

Hmm, I don't see any significant benefits of a generic
base::allocator::GetPropery with a string selector over the dedicated function.
I think it is possible to go with your suggestion. If we ever need another
function we can consider other options or return back to GetProperty way. I'll
implement the version you suggested.
Initially it was Waste instead of Internals, but James didn't like it. I took
the word Waste from the tcmalloc DumpStats report which btw also appears on the
chrome://tcmalloc/ page.

+      base::allocator::GetProperty("generic.current_allocated_bytes",
+                                   &allocated_bytes) &&
+      base::allocator::GetProperty("tcmalloc.pageheap_unmapped_bytes",
+                                   &unmapped_bytes)) {
+    *size = heap_size - allocated_bytes - unmapped_bytes;
+    return true;
+  }
+  return false;
+}
+
 void WebKitPlatformSupportImpl::SuspendSharedTimer() {
   ++shared_timer_suspended_;
 }
 
 void WebKitPlatformSupportImpl::ResumeSharedTimer() {
   // The shared timer may have fired or been adjusted while we were suspended.
   if (--shared_timer_suspended_ == 0 && !shared_timer_.IsRunning()) {
     setSharedTimerFireInterval(
         shared_timer_fire_time_ - monotonicallyIncreasingTime());
   }
@@ skipping 18 matching lines @@
   worker_task_runner->OnWorkerRunLoopStopped(runLoop);
 }
 
 #if defined(OS_ANDROID)
 WebKit::WebFlingAnimator* WebKitPlatformSupportImpl::createFlingAnimator() {
   return new FlingAnimatorImpl();
 }
 #endif
 
 }  // namespace webkit_glue