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

Unified Diff: third_party/WebKit/LayoutTests/virtual/feature-policy/http/tests/feature-policy/vibrate-enabledforself.php

Issue 2492623002: Implementation for feature policy - vibrate. (Closed)
Patch Set: Added layout tests for vibrate with runtime flag turned on 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/virtual/feature-policy/http/tests/feature-policy/vibrate-enabledforself.php
diff --git a/third_party/WebKit/LayoutTests/virtual/feature-policy/http/tests/feature-policy/vibrate-enabledforself.php b/third_party/WebKit/LayoutTests/virtual/feature-policy/http/tests/feature-policy/vibrate-enabledforself.php
new file mode 100644
index 0000000000000000000000000000000000000000..62bf8cc0357a95014a204a8db4f7673dfe414a0d
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/virtual/feature-policy/http/tests/feature-policy/vibrate-enabledforself.php
@@ -0,0 +1,18 @@
+<?php
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+// This test ensures that navigator.vibrate when enabled for self only works on
+// the same origin.
+
+Header("Feature-Policy: {\"vibrate\": [\"*\"]}");
iclelland 2016/11/18 04:46:48 Was this meant to be ["self"]?
lunalu1 2016/11/23 00:25:03 Done.
+?>
+
+<!DOCTYPE html>
+<script src="../../resources/testharness.js"></script>
+<script src="../../resources/testharnessreport.js"></script>
+<html>
+<iframe id="f1" src="resources/feature-policy-vibrate-enabledforself-same-origin.html">
iclelland 2016/11/18 04:46:48 Can we just use the feature-policy-vibrate-enabled
lunalu1 2016/11/23 00:25:03 Done.
+<iframe id="f2" src="http://localhost:8000/resources/feature-policy-vibrate-enabledforself-cross-origin.html">
iclelland 2016/11/18 04:46:48 Same here -- I think that we could use feature-pol
lunalu1 2016/11/23 00:25:03 Done.
+</html>

Powered by Google App Engine
This is Rietveld 408576698