Print malloc-ed memory when running with --trace-gc-object-stats.

src/base/accounting-allocator.h

 // Copyright 2016 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_BASE_ACCOUNTING_ALLOCATOR_H_
 #define V8_BASE_ACCOUNTING_ALLOCATOR_H_
 
 #include "src/base/atomicops.h"
 #include "src/base/macros.h"
 
 namespace v8 {
 namespace base {
 
-class AccountingAllocator final {
+class AccountingAllocator {
  public:
   AccountingAllocator() = default;
-  ~AccountingAllocator() = default;
+  virtual ~AccountingAllocator() = default;

[jochen (gone - plz use gerrit), 2016/09/05 10:00:26]

instead of making this virtual, you could have just moved the class out of base/
and added the tracking stuff to the allocator itself.

Not sure whether the additional checks are more expensive or paying for a
virtual lookup on every call...

 
   // Returns nullptr on failed allocation.
-  void* Allocate(size_t bytes);
-  void Free(void* memory, size_t bytes);
+  virtual void* Allocate(size_t bytes);
+  virtual void Free(void* memory, size_t bytes);
 
   size_t GetCurrentMemoryUsage() const;
   size_t GetMaxMemoryUsage() const;
 
  private:
   AtomicWord current_memory_usage_ = 0;
   AtomicWord max_memory_usage_ = 0;
 
   DISALLOW_COPY_AND_ASSIGN(AccountingAllocator);
 };
 
 }  // namespace base
 }  // namespace v8
 
 #endif  // V8_BASE_ACCOUNTING_ALLOCATOR_H_