Eliminate MediaPlayer & MediaPlayerClient abstractions

Source/core/testing/Internals.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  * Copyright (C) 2013 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 1684 matching lines @@
     if (!document->page()) {
         exceptionState.throwDOMException(InvalidAccessError, "No context document can be obtained.");
         return;
     }
     document->page()->setIsCursorVisible(isVisible);
 }
 
 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)
 {
     ASSERT(mediaElement);
     return mediaElement->effectiveMediaVolume();
 }
 
 void Internals::mediaPlayerRemoteRouteAvailabilityChanged(HTMLMediaElement* mediaElement, bool available)
 {
@@ skipping 609 matching lines @@
 {
     return frame()->selection().selectedTextForClipboard();
 }
 
 ValueIterable<int>::IterationSource* Internals::startIteration(ScriptState*, ExceptionState&)
 {
     return new InternalsIterationSource();
 }
 
 } // namespace blink