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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/permissions/resources/test-query.js

Issue 2458453002: [sensors] Add Permission guard to the generic sensor apis.
Patch Set: rebase + blink reformat 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
Index: third_party/WebKit/LayoutTests/http/tests/permissions/resources/test-query.js
diff --git a/third_party/WebKit/LayoutTests/http/tests/permissions/resources/test-query.js b/third_party/WebKit/LayoutTests/http/tests/permissions/resources/test-query.js
index fe16ccd5b08f617412692929c64cd9eac1a233c9..4e1669d41f688b2cc33bd1e43bcf5ea7dae31cc8 100644
--- a/third_party/WebKit/LayoutTests/http/tests/permissions/resources/test-query.js
+++ b/third_party/WebKit/LayoutTests/http/tests/permissions/resources/test-query.js
@@ -35,25 +35,41 @@ async_test(function(test) {
}, 'Test geolocation permission in ' + get_current_scope() + ' scope.');
async_test(function(test) {
- navigator.permissions.query({name:'midi'}).then(function(result) {
+ navigator.permissions.query({name: 'sensors'})
+ .then(function(result) {
+ assert_true(result instanceof PermissionStatus);
+ assert_equals(result.state, 'denied');
+ test.done();
+ })
+ .catch(function() {
+ assert_unreached('querying sensors permission should not fail.')
+ });
+}, 'Test sensors permission in ' + get_current_scope() + ' scope.');
+
+async_test(function(test) {
+ navigator.permissions.query({name: 'midi'})
+ .then(function(result) {
assert_true(result instanceof PermissionStatus);
assert_equals(result.state, 'denied');
// Test for sysex=false.
return navigator.permissions.query({name:'midi', sysex: false});
- }).then(function(result) {
+ })
+ .then(function(result) {
assert_true(result instanceof PermissionStatus);
assert_equals(result.state, 'denied');
// Test for sysex=true.
return navigator.permissions.query({name:'midi', sysex: true});
- }).then(function(result) {
+ })
+ .then(function(result) {
assert_true(result instanceof PermissionStatus);
assert_equals(result.state, 'denied');
test.done();
- }).catch(function() {
+ })
+ .catch(function() {
assert_unreached('querying midi permission should not fail.')
- });
+ });
}, 'Test midi permission in ' + get_current_scope() + ' scope.');
async_test(function(test) {

Powered by Google App Engine
This is Rietveld 408576698