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

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

Issue 2466333002: Automate Fullscreen web-platform-tests using wpt_automation (Closed)
Patch Set: update line numbers 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/fullscreen/api/document-fullscreen-element.html
diff --git a/third_party/WebKit/LayoutTests/fullscreen/api/document-fullscreen-element.html b/third_party/WebKit/LayoutTests/fullscreen/api/document-fullscreen-element.html
index 3e08e9a14a35dfdfb0c6cfc0963a7a2cbdee1158..03f11e5c9fb573bbbd0e71a09eb920bc6c882cf2 100644
--- a/third_party/WebKit/LayoutTests/fullscreen/api/document-fullscreen-element.html
+++ b/third_party/WebKit/LayoutTests/fullscreen/api/document-fullscreen-element.html
@@ -13,7 +13,8 @@ async_test(function(t)
{
assert_equals(document.fullscreenElement, div, "fullscreenElement before exitFullscreen()");
document.exitFullscreen();
- // FIXME: per spec fullscreenElement should still be div
+ // TODO(foolip): fullscreenElement should still be div.
+ // https://crbug.com/402421
assert_equals(document.fullscreenElement, null, "fullscreenElement after exitFullscreen()");
document.onfullscreenchange = t.step_func(function()
@@ -27,7 +28,8 @@ async_test(function(t)
{
assert_equals(document.fullscreenElement, null, "fullscreenElement before requestFullscreen()");
div.requestFullscreen();
- // FIXME: per spec fullscreenElement should still be null
+ // TODO(foolip): fullscreenElement should still be null.
+ // https://crbug.com/402421
assert_equals(document.fullscreenElement, div, "fullscreenElement after requestFullscreen()");
}), document.body);
});

Powered by Google App Engine
This is Rietveld 408576698