Refactoring: Unify WebMemoryAllocatorDump and WebMemoryAllocatorDumpImpl

third_party/WebKit/public/platform/WebMemoryAllocatorDump.h

 // Copyright 2015 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.
 
 #ifndef WebMemoryAllocatorDump_h
 #define WebMemoryAllocatorDump_h
 
 #include "WebCommon.h"
 #include "WebString.h"
 
+// TODO(hajimehoshi): WTF_MAKE_NONCOPYABLE works only in Blink side so far.
+// This macro should be replaced with DISALLOW_COPY_AND_ASSIGN in the near
+// future.
+#if INSIDE_BLINK
+#include "wtf/Noncopyable.h"
+#endif
+
+#include <stdint.h>
+
+namespace base {
+namespace trace_event {
+class MemoryAllocatorDump;
+} // namespace base
+} // namespace trace_event
+
 namespace blink {
 typedef uint64_t WebMemoryAllocatorDumpGuid;
 
 // A container which holds all the attributes of a particular dump for a given
 // allocator.
 class BLINK_PLATFORM_EXPORT WebMemoryAllocatorDump {
+#if INSIDE_BLINK
+    WTF_MAKE_NONCOPYABLE(WebMemoryAllocatorDump);

[esprehn, 2016/02/29 06:51:20]

Don't do this. Use WebNoncopyable which does what you want.

[haraken, 2016/02/29 06:55:30]

So as tkent@ and I are saying in the above, I'd suggest simply removing
NONCOPYABLE things from this CL. That's a separate issue and it should be
addressed in a separate CL.

[hajimehoshi, 2016/02/29 08:21:48]

Done.

+#endif
 public:
-    virtual ~WebMemoryAllocatorDump();
+    explicit WebMemoryAllocatorDump(base::trace_event::MemoryAllocatorDump*);
+    ~WebMemoryAllocatorDump();
 
     // Adds a scalar attribute to the dump.
     // Arguments:
     //   name: name of the attribute. Typical names, emitted by most allocators
     //       dump providers are: "size" and "objects_count".
     //   units: the units for the attribute. Gives a hint to the trace-viewer UI
     //       about the semantics of the attribute.
     //       Currently supported values are "bytes" and "objects".
     //   value: the value of the attribute.
-    virtual void addScalar(const char* name, const char* units, uint64_t value) {}
-    virtual void addScalarF(const char* name, const char* units, double value) {}
-    virtual void addString(const char* name, const char* units, const WebString& value) {}
+    void addScalar(const char* name, const char* units, uint64_t value);
+    void addScalarF(const char* name, const char* units, double value);
+    void addString(const char* name, const char* units, const WebString& value);
 
     // |guid| is an optional global dump identifier, unique across all processes
     // within the scope of a global dump. It is only required when using the
     // graph APIs (see AddOwnershipEdge) to express retention / suballocation or
     // cross process sharing. See crbug.com/492102 for design docs.
     // Subsequent MemoryAllocatorDump(s) with the same |absolute_name| are
     // expected to have the same guid.
-    virtual WebMemoryAllocatorDumpGuid guid() const
-    {
-        BLINK_ASSERT_NOT_REACHED();
-        return 0;
-    }
+    WebMemoryAllocatorDumpGuid guid() const;
+
+private:
+    base::trace_event::MemoryAllocatorDump* m_memoryAllocatorDump; // Not owned.
+    blink::WebMemoryAllocatorDumpGuid m_guid;
 };
 
 } // namespace blink
 
 #endif // WebMemoryAllocatorDump_h