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

Unified Diff: third_party/WebKit/LayoutTests/imported/wpt/fullscreen/api/document-fullscreen-element-manual.html

Issue 2463743003: Import web-platform-tests manual tests by default (Closed)
Patch Set: update deps_updater_unittest.py Created 4 years, 1 month 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/imported/wpt/fullscreen/api/document-fullscreen-element-manual.html
diff --git a/third_party/WebKit/LayoutTests/fullscreen/api/document-fullscreen-element.html b/third_party/WebKit/LayoutTests/imported/wpt/fullscreen/api/document-fullscreen-element-manual.html
similarity index 68%
copy from third_party/WebKit/LayoutTests/fullscreen/api/document-fullscreen-element.html
copy to third_party/WebKit/LayoutTests/imported/wpt/fullscreen/api/document-fullscreen-element-manual.html
index 3e08e9a14a35dfdfb0c6cfc0963a7a2cbdee1158..473bdbb0f6174971d75425eea0e6f51595d6e4aa 100644
--- a/third_party/WebKit/LayoutTests/fullscreen/api/document-fullscreen-element.html
+++ b/third_party/WebKit/LayoutTests/imported/wpt/fullscreen/api/document-fullscreen-element-manual.html
@@ -1,8 +1,8 @@
<!DOCTYPE html>
<title>Document.fullscreenElement</title>
-<script src="../../resources/testharness.js"></script>
-<script src="../../resources/testharnessreport.js"></script>
-<script src="../trusted-event.js"></script>
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="../trusted-click.js"></script>
<div id="log"></div>
<script>
async_test(function(t)
@@ -13,8 +13,7 @@ async_test(function(t)
{
assert_equals(document.fullscreenElement, div, "fullscreenElement before exitFullscreen()");
document.exitFullscreen();
- // FIXME: per spec fullscreenElement should still be div
- assert_equals(document.fullscreenElement, null, "fullscreenElement after exitFullscreen()");
+ assert_equals(document.fullscreenElement, div, "fullscreenElement after exitFullscreen()");
document.onfullscreenchange = t.step_func(function()
{
@@ -23,12 +22,11 @@ async_test(function(t)
});
});
- trusted_event(t.step_func(function()
+ trusted_click(t.step_func(function()
{
assert_equals(document.fullscreenElement, null, "fullscreenElement before requestFullscreen()");
div.requestFullscreen();
- // FIXME: per spec fullscreenElement should still be null
- assert_equals(document.fullscreenElement, div, "fullscreenElement after requestFullscreen()");
+ assert_equals(document.fullscreenElement, null, "fullscreenElement after requestFullscreen()");
}), document.body);
});
</script>

Powered by Google App Engine
This is Rietveld 408576698