Chromium Code Reviews| Index: Source/core/dom/FullscreenElementStack.cpp |
| diff --git a/Source/core/dom/FullscreenElementStack.cpp b/Source/core/dom/FullscreenElementStack.cpp |
| index 046d275e54b97e9bcad3b7e968d98e482da2e0be..2ef4d0a01fcf9b3f48882dccd505f4a0d5f00076 100644 |
| --- a/Source/core/dom/FullscreenElementStack.cpp |
| +++ b/Source/core/dom/FullscreenElementStack.cpp |
| @@ -35,6 +35,7 @@ |
| #include "core/frame/LocalFrame.h" |
| #include "core/frame/Settings.h" |
| #include "core/html/HTMLFrameOwnerElement.h" |
| +#include "core/html/HTMLMediaElement.h" |
| #include "core/page/Chrome.h" |
| #include "core/page/ChromeClient.h" |
| #include "core/rendering/RenderFullScreen.h" |
| @@ -189,7 +190,7 @@ void FullscreenElementStack::requestFullScreenForElement(Element* element, unsig |
| // - an activation behavior is currently being processed whose click event was trusted, or |
| // - the event listener for a trusted click event is being handled. |
| // FIXME: Does this need to null-check settings()? |
| - if (!UserGestureIndicator::processingUserGesture() && (!element->isMediaElement() || document()->settings()->mediaFullscreenRequiresUserGesture())) |
| + if (!UserGestureIndicator::processingUserGesture() && (!isHTMLMediaElement(element) || document()->settings()->mediaFullscreenRequiresUserGesture())) |
|
Inactive
2014/03/13 18:46:24
Ditto
|
| break; |
| // There is a previously-established user preference, security risk, or platform limitation. |