New media playback UI.

Source/core/html/HTMLMediaElement.cpp

Index: Source/core/html/HTMLMediaElement.cpp
diff --git a/Source/core/html/HTMLMediaElement.cpp b/Source/core/html/HTMLMediaElement.cpp
index e905c8905d3cdb9f0a85e36d437d9eaacd134618..ca5f37366deae7a7851b20008fd103c2940268cf 100644
--- a/Source/core/html/HTMLMediaElement.cpp
+++ b/Source/core/html/HTMLMediaElement.cpp
@@ -3055,7 +3055,7 @@ void HTMLMediaElement::clearMediaPlayer(int flags)
     m_remoteRoutesAvailable = false;
     m_playingRemotely = false;
     if (mediaControls())
-        mediaControls()->refreshCastButtonVisibility();
+        mediaControls()->refreshCastButtonVisibilityWithoutUpdate();
 
     if (m_textTracks)
         configureTextTrackDisplay(AssumeNoVisibleChange);
@@ -3584,6 +3584,12 @@ bool HTMLMediaElement::isBlockedOnMediaController() const
     return false;
 }
 
+void HTMLMediaElement::setAllowHiddenAudioElements(bool allow)
+{
+    if (mediaControls())

[philipj_slow, 2015/07/14 11:16:22]

ensureMediaControls() instead of the null check would probably be good, so that
the caller doesn't need to take care to add the controls attribute before using
this API.

[liberato (no reviews please), 2015/07/14 22:10:37]

done.  i chose the null check because it's unclear what this function should do
if there are no controls.  i considered an assert, since it's a test hook.

-fl

+        mediaControls()->setAllowHiddenAudioElements(allow);
+}
+
 WebMediaPlayer::CORSMode HTMLMediaElement::corsMode() const
 {
     const AtomicString& crossOriginMode = fastGetAttribute(crossoriginAttr);