Refactoring: Move some classes from content/child to platform

third_party/WebKit/public/platform/Platform.h

Index: third_party/WebKit/public/platform/Platform.h
diff --git a/third_party/WebKit/public/platform/Platform.h b/third_party/WebKit/public/platform/Platform.h
index 784aa7eb0fb74ccdaaffcba5f33e85c7d258468d..92d33fc1c55ba8ee9686e9fa0aca11095d3341f9 100644
--- a/third_party/WebKit/public/platform/Platform.h
+++ b/third_party/WebKit/public/platform/Platform.h
@@ -458,10 +458,10 @@ public:
     // registerMemoryDumpProvider() method. |name| is used for debugging
     // (duplicates are allowed) and must be a long-lived C string.
     // See crbug.com/458295 for design docs.
-    virtual void registerMemoryDumpProvider(blink::WebMemoryDumpProvider*, const char* name) { }
+    BLINK_PLATFORM_EXPORT void registerMemoryDumpProvider(blink::WebMemoryDumpProvider*, const char* name);

[kinuko, 2016/02/04 13:15:54]

If these are not called outside blink I don't think they should be in public/;
please move them from public/platform to Source/platform.

Also I don't think they need to be kept as Platform:: methods, they could be
just moved to a separate utility file in Source/platform, couldn't they?  (I
haven't looked at the callsite yet though)

[hajimehoshi, 2016/02/05 07:41:54]

Done.

[hajimehoshi, 2016/02/05 07:41:54]

I've just now found that TestBlinkWebUnitTestSupport, which is another class
inheriting Platform, must have empty (un)registerMemoryDumpProvider
implementation . This means The actual implementation BlinkPlatformImpl in
Chromium side should call this logic in Blink. I'll change their actual
implementation as static functions and leave BLINK_PLATFORM_EXPORT for the new
functions. Sorry for the confusion.

 
     // Must be called on the thread that called registerMemoryDumpProvider().
-    virtual void unregisterMemoryDumpProvider(blink::WebMemoryDumpProvider*) { }
+    BLINK_PLATFORM_EXPORT void unregisterMemoryDumpProvider(blink::WebMemoryDumpProvider*);
 
     class TraceLogEnabledStateObserver {
     public:
@@ -474,9 +474,6 @@ public:
     virtual void addTraceLogEnabledStateObserver(TraceLogEnabledStateObserver*) {}
     virtual void removeTraceLogEnabledStateObserver(TraceLogEnabledStateObserver*) {}
 
-    // Returns a newly allocated WebProcessMemoryDump instance.
-    virtual blink::WebProcessMemoryDump* createProcessMemoryDump() { return nullptr; }
-
     typedef uint64_t WebMemoryAllocatorDumpGuid;
 
     // Returns guid corresponding to the given string (the hash value) for