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

Unified Diff: LayoutTests/fullscreen/enter-exit-full-screen-hover.html

Issue 552903004: Update hover state after entering and leaving fullscreen. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Added layouttest Created 6 years, 3 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 | « no previous file | LayoutTests/fullscreen/enter-exit-full-screen-hover-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/fullscreen/enter-exit-full-screen-hover.html
diff --git a/LayoutTests/fullscreen/enter-exit-full-screen-hover.html b/LayoutTests/fullscreen/enter-exit-full-screen-hover.html
new file mode 100644
index 0000000000000000000000000000000000000000..503ed267a07b32058e6acce42ac240f552755ce7
--- /dev/null
+++ b/LayoutTests/fullscreen/enter-exit-full-screen-hover.html
@@ -0,0 +1,57 @@
+<!DOCTYPE html>
+<script src="../resources/js-test.js"></script>
+<script src="full-screen-test.js"></script>
+<script src="resources/touch-hover-active-tests.js"></script>
+<link rel="stylesheet" href="resources/touch-hover-active-tests.css">
+
+<button id="enter" class='touch-interactive' onclick="document.body.webkitRequestFullScreen()">Go full screen</button>
+<button id="exit" class='touch-interactive' onclick="document.webkitCancelFullScreen()">Exit full screen</button>
+
+<script>
+var enterButton;
+var exitButton;
+
+function runTest() {
+ if (!window.eventSender) {
+ debug('This test requires DRT.');
+ return;
+ }
+
+ if (!eventSender.gestureShowPress || !eventSender.gestureTap || !eventSender.mouseMoveTo) {
philipj_slow 2014/09/19 14:31:01 This looks like it was copy-pasted, only mouseMove
xun.sun 2014/09/20 17:39:48 Done.
+ debug('GestureShowPress, GestureTap, or MouseMoveTo is not supported by this platform');
+ return;
+ }
+
+ enterButton = document.getElementById('enter');
+ exitButton = document.getElementById('exit');
+
+ var center = elementCenter(enterButton);
+
+ waitForEventOnce(document, 'webkitfullscreenchange', function() {
+ test("document.webkitIsFullScreen==true")
philipj_slow 2014/09/19 14:31:02 Please use shouldBe* instead of test: - te
xun.sun 2014/09/20 17:39:48 Done.
+ // After entering fullscreen, the button should lose hover
+ test("getHoverActiveState(enterButton)=='default'")
+
+ waitForEventOnce(document, 'webkitfullscreenchange', function() {
+ test("document.webkitIsFullScreen==false")
+ // After leaving fullscreen, the button should lose hover
+ test("getHoverActiveState(exitButton)=='default'")
+ endTest();
+ });
+
+ var center_1 = elementCenter(exitButton);
philipj_slow 2014/09/19 14:31:02 How about enterButtonCenter and exitButtonCenter i
xun.sun 2014/09/20 17:39:48 Done.
+ // Hover on and click the "Exit fullscreen" button
+ eventSender.mouseMoveTo(center_1.x, center_1.y);
+ eventSender.mouseDown();
+ eventSender.mouseUp();
+ });
+
+
+ // Move mouse over to the "Go fullscreen" button and click
+ eventSender.mouseMoveTo(center.x, center.y);
+ eventSender.mouseDown();
+ eventSender.mouseUp();
+}
+
+runTest();
+</script>
« no previous file with comments | « no previous file | LayoutTests/fullscreen/enter-exit-full-screen-hover-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698