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

Unified Diff: LayoutTests/fullscreen/api/element-request-fullscreen-non-top.html

Issue 482543002: Ignore fullscreen requests for the current fullscreen element (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: rebase Created 6 years, 4 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/api/element-request-fullscreen-top.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/fullscreen/api/element-request-fullscreen-non-top.html
diff --git a/LayoutTests/fullscreen/model/remove-last.html b/LayoutTests/fullscreen/api/element-request-fullscreen-non-top.html
similarity index 71%
copy from LayoutTests/fullscreen/model/remove-last.html
copy to LayoutTests/fullscreen/api/element-request-fullscreen-non-top.html
index b7a3807193caf94240c1f5532edc0977ccc2cb6b..3a5c657c6a278e6145f42783e0371193882519a5 100644
--- a/LayoutTests/fullscreen/model/remove-last.html
+++ b/LayoutTests/fullscreen/api/element-request-fullscreen-non-top.html
@@ -1,5 +1,5 @@
<!DOCTYPE html>
-<title>Remove the last element on the fullscreen element stack</title>
+<title>Element.requestFullscreen() for non-top element in fullscreen element stack</title>
<script src="../../resources/testharness.js"></script>
<script src="../../resources/testharnessreport.js"></script>
<script src="../trusted-event.js"></script>
@@ -20,12 +20,8 @@ async_test(function(t)
document.onfullscreenchange = t.step_func(function()
{
assert_equals(document.fullscreenElement, last);
- last.remove();
- document.onfullscreenchange = t.step_func(function()
- {
- assert_equals(document.fullscreenElement, first);
- t.done();
- });
+ trusted_request(first, last);
+ document.onfullscreenerror = t.step_func_done();
});
});
});
« no previous file with comments | « no previous file | LayoutTests/fullscreen/api/element-request-fullscreen-top.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698