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

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

Issue 1363023005: Implement FullScreen using top layer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 2 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 1aa60aabd5f07469beb55a5a9fc2c54bd2fee042..be3e258bae26206a64d6d219754e8f7ce08a6fd1 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
@@ -1,4 +1,11 @@
<!DOCTYPE html>
+<style>
+/* Offset body by 50px so that buttons clearly change position when they enter/exit fullscreen. */
+body {
+ position: relative;
+ left: 50px;
+}
+</style>
<script src="../resources/js-test.js"></script>
<script src="full-screen-test.js"></script>
<script src="../fast/events/touch/resources/touch-hover-active-tests.js"></script>
@@ -43,7 +50,8 @@ function runTest() {
// Move mouse over to the "Go fullscreen" button and click
- eventSender.mouseMoveTo(enterButtonCenter.x, enterButtonCenter.y);
+ // We move by 50px in the x-direction to account for the relative position.
+ eventSender.mouseMoveTo(enterButtonCenter.x + 50, enterButtonCenter.y);
eventSender.mouseDown();
eventSender.mouseUp();
}

Powered by Google App Engine
This is Rietveld 408576698