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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: third_party/WebKit/LayoutTests/vibration/vibration-patterns.html

Issue 2778693004: Remove navigator.vibrate without user gesture. (Closed)
Patch Set: rebase Created 3 years, 8 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 | « third_party/WebKit/LayoutTests/vibration/vibration-page-visibility.html ('k') | third_party/WebKit/Source/core/frame/Deprecation.cpp » ('j') | third_party/WebKit/Source/core/frame/Deprecation.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/vibration/vibration-patterns.html
diff --git a/third_party/WebKit/LayoutTests/vibration/vibration-patterns.html b/third_party/WebKit/LayoutTests/vibration/vibration-patterns.html
index ca254ea8a17ca655f852b45b6328d74ec9848224..75aa0f57993976bc4dab8b184c75ad3286bebdfc 100644
--- a/third_party/WebKit/LayoutTests/vibration/vibration-patterns.html
+++ b/third_party/WebKit/LayoutTests/vibration/vibration-patterns.html
@@ -1,12 +1,18 @@
<html>
<head>
<script src="../resources/js-test.js"></script>
+<script src="../resources/user-gesture-utils.js"></script>
<script src="vibration-utils.js"></script>
</head>
<body>
+<h4 id="test"></h4>
<script>
description('Tests for how patterns are handled in the Vibration API.');
+// Simulates a user click for vibrate to be allowed.
+var element = document.getElementById("test");
+simulateUserClick(element.offsetLeft + 2, element.offsetTop + 2);
+
// A trailing pause is discarded from a pattern i.e. patterns of even
// non-zero length are truncated by 1.
shouldBeTrue("navigator.vibrate(newPattern(1, 4))");
« no previous file with comments | « third_party/WebKit/LayoutTests/vibration/vibration-page-visibility.html ('k') | third_party/WebKit/Source/core/frame/Deprecation.cpp » ('j') | third_party/WebKit/Source/core/frame/Deprecation.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698