Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(973)

Unified Diff: Source/core/html/HTMLMediaElement.cpp

Issue 13851023: Remove ChromeClient cruft (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Patch for landing Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/dom/Document.cpp ('k') | Source/core/html/HTMLVideoElement.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/html/HTMLMediaElement.cpp
diff --git a/Source/core/html/HTMLMediaElement.cpp b/Source/core/html/HTMLMediaElement.cpp
index 2dbb1eedee12b23a3dd1a26d1820dd68a6c93472..d10cef866debf70f1116c25487f2dcb35cc10bfc 100644
--- a/Source/core/html/HTMLMediaElement.cpp
+++ b/Source/core/html/HTMLMediaElement.cpp
@@ -2394,10 +2394,6 @@ bool HTMLMediaElement::controls() const
if (frame && !frame->script()->canExecuteScripts(NotAboutToExecuteScript))
return true;
- // always show controls for video when fullscreen playback is required.
- if (isVideo() && document()->page() && document()->page()->chrome()->requiresFullscreenForVideoPlayback())
- return true;
-
// Always show controls when in full screen mode.
if (isFullscreen())
return true;
@@ -3582,9 +3578,6 @@ void HTMLMediaElement::updatePlayState()
invalidateCachedTime();
if (playerPaused) {
- if (!m_isFullscreen && isVideo() && document() && document()->page() && document()->page()->chrome()->requiresFullscreenForVideoPlayback())
- enterFullscreen();
-
// Set rate, muted before calling play in case they were set before the media engine was setup.
// The media engine should just stash the rate and muted values since it isn't already playing.
m_player->setRate(m_playbackRate);
@@ -3830,8 +3823,6 @@ void HTMLMediaElement::exitFullscreen()
if (hasMediaControls())
mediaControls()->exitedFullscreen();
if (document() && document()->page()) {
- if (document()->page()->chrome()->requiresFullscreenForVideoPlayback())
- pauseInternal();
document()->page()->chrome()->client()->exitFullscreenForNode(this);
scheduleEvent(eventNames().webkitendfullscreenEvent);
}
« no previous file with comments | « Source/core/dom/Document.cpp ('k') | Source/core/html/HTMLVideoElement.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698