Eliminate MediaPlayer & MediaPlayerClient abstractions

Source/core/testing/Internals.cpp

Index: Source/core/testing/Internals.cpp
diff --git a/Source/core/testing/Internals.cpp b/Source/core/testing/Internals.cpp
index f0c308a4aa384e11cbd4b007701a7e48980ceb96..5c938233d25f14cba2a951f3bb4c1d035a5ba287 100644
--- a/Source/core/testing/Internals.cpp
+++ b/Source/core/testing/Internals.cpp
@@ -1702,7 +1702,7 @@ void Internals::setIsCursorVisible(Document* document, bool isVisible, Exception
 void Internals::mediaPlayerRequestFullscreen(HTMLMediaElement* mediaElement)
 {
     ASSERT(mediaElement);
-    mediaElement->mediaPlayerRequestFullscreen();
+    mediaElement->requestFullscreen();

[philipj_slow, 2015/04/14 08:42:54]

What is this change for? Does
LayoutTests/media/media-player-request-fullscreen.html still pass? If so that
test must be broken because it should verify that it goes fullscreen even
without a user gesture.

[Srirama, 2015/04/14 12:09:28]

This is just renaming of the API (mediaPlayerRequestFullscreen is renamed to
requestFullscreen as HTMLMediaElement is now implementing WebMediaPlayerClient
instead of MediaPlayerClient)

 }
 
 double Internals::effectiveMediaVolume(HTMLMediaElement* mediaElement)