Eliminate MediaPlayer & MediaPlayerClient abstractions(play/pause, other APIs)

Source/core/html/HTMLMediaElement.h

Index: Source/core/html/HTMLMediaElement.h
diff --git a/Source/core/html/HTMLMediaElement.h b/Source/core/html/HTMLMediaElement.h
index 3561923f765376bdb21bb63c0be635c4f16354d4..f348661c36f8d6c95a168ed3b1879ec4fa59ec4a 100644
--- a/Source/core/html/HTMLMediaElement.h
+++ b/Source/core/html/HTMLMediaElement.h
@@ -234,7 +234,7 @@ public:
     // of one of them here.
     virtual ExecutionContext* executionContext() const OVERRIDE FINAL { return HTMLElement::executionContext(); }
 
-    bool hasSingleSecurityOrigin() const { return !m_player || m_player->hasSingleSecurityOrigin(); }
+    bool hasSingleSecurityOrigin() const { return webMediaPlayer() && webMediaPlayer()->hasSingleSecurityOrigin(); }

[philipj_slow, 2014/05/27 20:54:56]

Did you deliberately change the return value when the player is null? Because of
how WebGLRenderingContextBase::validateHTMLVideoElement is implemented it should
be impossible to hit that case here, so I think it's OK.

 
     bool isFullscreen() const;
     void enterFullscreen();