Index: Source/core/html/HTMLMediaElement.cpp |
diff --git a/Source/core/html/HTMLMediaElement.cpp b/Source/core/html/HTMLMediaElement.cpp |
index 0f89c37d54bfecc8a6533843fbee051e2f78930d..8b3a910f7e5fb4775332586cbe37592eb383b73b 100644 |
--- a/Source/core/html/HTMLMediaElement.cpp |
+++ b/Source/core/html/HTMLMediaElement.cpp |
@@ -2849,6 +2849,11 @@ void HTMLMediaElement::mediaPlayerPlaybackStateChanged() |
void HTMLMediaElement::mediaPlayerRequestFullscreen() |
{ |
WTF_LOG(Media, "HTMLMediaElement::mediaPlayerRequestFullscreen"); |
+ |
+ // The player is responsible for only invoking this callback in response to |
+ // user interaction or when it is technically required to play the video. |
+ UserGestureIndicator gestureIndicator(DefinitelyProcessingNewUserGesture); |
qinmin
2014/03/24 06:32:54
will the gesture be consumed by FullscreenElementS
philipj_slow
2014/03/24 15:18:23
The UserGestureIndicator is only active while it's
acolwell GONE FROM CHROMIUM
2014/03/25 22:14:43
How about we just move the settings code from Full
philipj_slow
2014/03/26 07:48:17
Sorry, I didn't communicate my intentions very cle
acolwell GONE FROM CHROMIUM
2014/03/26 17:39:43
Ok. Given the other 2 CLs, I'm fine with doing def
|
+ |
enterFullscreen(); |
} |