Title: [109018] trunk/LayoutTests
Revision
109018
Author
commit-qu...@webkit.org
Date
2012-02-27 13:10:52 -0800 (Mon, 27 Feb 2012)

Log Message

Cleanup Gesture requirement when exiting video-play-require-user-gesture test
https://bugs.webkit.org/show_bug.cgi?id=79690

Patch by Min Qin <qin...@google.com> on 2012-02-27
Reviewed by Philippe Normand.

* media/video-play-require-user-gesture.html:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (109017 => 109018)


--- trunk/LayoutTests/ChangeLog	2012-02-27 21:03:16 UTC (rev 109017)
+++ trunk/LayoutTests/ChangeLog	2012-02-27 21:10:52 UTC (rev 109018)
@@ -1,3 +1,12 @@
+2012-02-27  Min Qin  <qin...@google.com>
+
+        Cleanup Gesture requirement when exiting video-play-require-user-gesture test
+        https://bugs.webkit.org/show_bug.cgi?id=79690
+
+        Reviewed by Philippe Normand.
+
+        * media/video-play-require-user-gesture.html:
+
 2012-02-27  Mihnea Ovidenie  <mih...@adobe.com>
 
         [CSSRegions]-webkit-flow-into initial value should be none instead of auto

Modified: trunk/LayoutTests/media/video-play-require-user-gesture.html (109017 => 109018)


--- trunk/LayoutTests/media/video-play-require-user-gesture.html	2012-02-27 21:03:16 UTC (rev 109017)
+++ trunk/LayoutTests/media/video-play-require-user-gesture.html	2012-02-27 21:10:52 UTC (rev 109018)
@@ -9,6 +9,11 @@
             if (window.internals) 
                 window.internals.setMediaPlaybackRequiresUserGesture(document, true);
 
+            function cleanGestureRequirement() {
+                if (window.internals) 
+                    window.internals.setMediaPlaybackRequiresUserGesture(document, false);
+            }
+
             function click()
             {
                 if (window.eventSender) {
@@ -16,6 +21,7 @@
                     try {
                         playCoords = mediaControlsButtonCoordinates(video, "play-button");
                     } catch (exception) {
+                        cleanGestureRequirement();
                         failTest(exception.description);
                         return;
                     }
@@ -32,6 +38,7 @@
             function playing()
             {
                 if (userGestureInitiated == 0) {
+                    cleanGestureRequirement();
                     failTest("Should not play without user gesture.");
                 } else {
                     run("video.pause()");
@@ -47,8 +54,10 @@
                 try {
                     run("video.webkitEnterFullScreen()");
                 } catch(ex) {
+                    cleanGestureRequirement();
                     failTest("video.webkitEnterFullScreen() still requires user gesture.");
                 }
+                cleanGestureRequirement(); 
                 endTest();
             }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to