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

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

Issue 2654083006: Revert of Sync requestFullscreen() and exitFullscreen() algorithms with the spec (Closed)
Patch Set: add failing test expectations Created 3 years, 11 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
Index: third_party/WebKit/LayoutTests/fullscreen/enter-exit-full-screen-hover.html
diff --git a/third_party/WebKit/LayoutTests/fullscreen/enter-exit-full-screen-hover.html b/third_party/WebKit/LayoutTests/fullscreen/enter-exit-full-screen-hover.html
index 85f4a279abe616cbc70f93a636ce0a32a546d226..1aa60aabd5f07469beb55a5a9fc2c54bd2fee042 100644
--- a/third_party/WebKit/LayoutTests/fullscreen/enter-exit-full-screen-hover.html
+++ b/third_party/WebKit/LayoutTests/fullscreen/enter-exit-full-screen-hover.html
@@ -23,30 +23,22 @@ function runTest() {
var enterButtonCenter = elementCenter(enterButton);
waitForEventOnce(document, 'webkitfullscreenchange', function() {
- shouldBeTrue("document.webkitIsFullScreen");
- // After entering fullscreen + layout, the button should lose hover.
- // TODO(foolip): Synchronize hover state changes with animation frames.
- // https://crbug.com/668758
- shouldBeOnlyHovered("getHoverActiveState(enterButton)");
- testRunner.layoutAndPaintAsyncThen(function() {
- shouldBeDefault("getHoverActiveState(enterButton)");
-
- waitForEventOnce(document, 'webkitfullscreenchange', function() {
- shouldBeFalse("document.webkitIsFullScreen");
- // After exiting fullscreen + layout, the button should lose hover.
- shouldBeOnlyHovered("getHoverActiveState(exitButton)");
- testRunner.layoutAndPaintAsyncThen(function() {
- shouldBeDefault("getHoverActiveState(exitButton)");
- endTest();
- });
- });
-
- var exitButtonCenter = elementCenter(exitButton);
- // Hover on and click the "Exit fullscreen" button
- eventSender.mouseMoveTo(exitButtonCenter.x, exitButtonCenter.y);
- eventSender.mouseDown();
- eventSender.mouseUp();
+ shouldBeTrue("document.webkitIsFullScreen")
+ // After entering fullscreen, the button should lose hover
+ shouldBeDefault("getHoverActiveState(enterButton)")
+
+ waitForEventOnce(document, 'webkitfullscreenchange', function() {
+ shouldBeFalse("document.webkitIsFullScreen")
+ // After leaving fullscreen, the button should lose hover
+ shouldBeDefault("getHoverActiveState(exitButton)")
+ endTest();
});
+
+ var exitButtonCenter = elementCenter(exitButton);
+ // Hover on and click the "Exit fullscreen" button
+ eventSender.mouseMoveTo(exitButtonCenter.x, exitButtonCenter.y);
+ eventSender.mouseDown();
+ eventSender.mouseUp();
});

Powered by Google App Engine
This is Rietveld 408576698