Oilpan: move the MediaPlayer and MediaPlayerClient objects to the heap.

Source/platform/graphics/media/MediaPlayer.h

Index: Source/platform/graphics/media/MediaPlayer.h
diff --git a/Source/platform/graphics/media/MediaPlayer.h b/Source/platform/graphics/media/MediaPlayer.h
index a4d7ccdcac9f7590937e4603dd97d7403416489c..064aad205a3ad71e121ef27dd83ac69169b168cc 100644
--- a/Source/platform/graphics/media/MediaPlayer.h
+++ b/Source/platform/graphics/media/MediaPlayer.h
@@ -28,6 +28,7 @@
 
 #include "platform/PlatformExport.h"
 #include "platform/graphics/GraphicsTypes3D.h"
+#include "platform/heap/Handle.h"
 #include "public/platform/WebMediaPlayer.h"
 #include "wtf/Forward.h"
 #include "wtf/Noncopyable.h"
@@ -55,10 +56,12 @@ class TimeRanges;
 typedef unsigned GC3Denum;
 typedef int GC3Dint;
 
-class MediaPlayerClient {
+class MediaPlayerClient : public WillBeGarbageCollectedMixin {

[sof, 2014/06/02 11:36:28]

The use of DummyBase<void> breaks the component build in the non-Oilpan case,
see
http://build.chromium.org/p/tryserver.blink/builders/win_blink_compile_dbg/bu...

I _thought_ this was due to DummyBase being provided via WebCore & moved it to
WTF instead, but no. I've not yet #if-ed my way around the issue..is there a
better way to handle it?

 public:
     virtual ~MediaPlayerClient() { }
 
+    virtual void trace(Visitor*) { }
+
     // the network state has changed
     virtual void mediaPlayerNetworkStateChanged() = 0;
 
@@ -97,12 +100,12 @@ public:
     virtual void mediaPlayerMediaSourceOpened(blink::WebMediaSource*) = 0;
 };
 
-typedef PassOwnPtr<MediaPlayer> (*CreateMediaEnginePlayer)(MediaPlayerClient*);
+typedef PassOwnPtrWillBeRawPtr<MediaPlayer> (*CreateMediaEnginePlayer)(MediaPlayerClient*);
 
-class PLATFORM_EXPORT MediaPlayer {
+class PLATFORM_EXPORT MediaPlayer : public WillBeGarbageCollectedMixin {
     WTF_MAKE_NONCOPYABLE(MediaPlayer);
 public:
-    static PassOwnPtr<MediaPlayer> create(MediaPlayerClient*);
+    static PassOwnPtrWillBeRawPtr<MediaPlayer> create(MediaPlayerClient*);
     static void setMediaEngineCreateFunction(CreateMediaEnginePlayer);
 
     static double invalidTime() { return -1.0; }
@@ -110,6 +113,10 @@ public:
     MediaPlayer() { }
     virtual ~MediaPlayer() { }
 
+    virtual void trace(Visitor*) { }
+
+    virtual void detach() = 0;
+
     virtual void load(blink::WebMediaPlayer::LoadType, const String& url, blink::WebMediaPlayer::CORSMode) = 0;
 
     virtual void play() = 0;