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

Unified Diff: third_party/WebKit/LayoutTests/fast/media/mq-hover.html

Issue 2673963002: Drop hover:on-demand support & let touch-screens report "none" instead. (Closed)
Patch Set: Fixed a "yikes!" Created 3 years, 9 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/fast/media/mq-hover.html
diff --git a/third_party/WebKit/LayoutTests/fast/media/mq-hover.html b/third_party/WebKit/LayoutTests/fast/media/mq-hover.html
index 2fae6804e028db30f1dcb922f2ecee181e495caf..d19c3df241eb5e25bf9b294ffeed6cbf18503970 100644
--- a/third_party/WebKit/LayoutTests/fast/media/mq-hover.html
+++ b/third_party/WebKit/LayoutTests/fast/media/mq-hover.html
@@ -20,7 +20,6 @@
"(hover:none)",
"(hover:NoNe)",
"(hover:on-demand)",
- "(hover:oN-dEmAnd)",
"(hover:bogusvalue)",
"(hover:hover)",
"(hover:HoveR)",
@@ -29,7 +28,6 @@
"(any-hover:none)",
"(any-hover:NoNe)",
"(any-hover:on-demand)",
- "(any-hover:oN-dEmAnd)",
"(any-hover:bogusvalue)",
"(any-hover:hover)",
"(any-hover:HoveR)"
@@ -45,29 +43,14 @@
if (!window.internals)
return;
- log("------------- with 'on-demand' hover -------------");
- internals.settings.setPrimaryHoverType('on-demand');
- internals.settings.setAvailableHoverTypes('on-demand');
- testQueries();
-
log("------------- with 'hover' hover -------------");
internals.settings.setPrimaryHoverType('hover');
internals.settings.setAvailableHoverTypes('hover');
testQueries();
- log("------------- with 'on-demand' and 'hover' hover, 'hover' is primary -------------");
- internals.settings.setPrimaryHoverType('hover');
- internals.settings.setAvailableHoverTypes('on-demand,hover');
- testQueries();
-
- log("------------- with 'on-demand' and 'hover' hover, 'on-demand' is primary -------------");
- internals.settings.setPrimaryHoverType('on-demand');
- internals.settings.setAvailableHoverTypes('on-demand,hover');
- testQueries();
-
- log("------------- with 'none', 'on-demand' and 'hover' hover, 'none' is primary -------------");
+ log("------------- with 'none' and 'hover' hover, 'none' is primary -------------");
internals.settings.setPrimaryHoverType('none');
- internals.settings.setAvailableHoverTypes('none,on-demand,hover');
+ internals.settings.setAvailableHoverTypes('none,hover');
testQueries();
log("------------- with 'none' hover -------------");
« no previous file with comments | « content/public/common/web_preferences.cc ('k') | third_party/WebKit/LayoutTests/fast/media/mq-hover-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698