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

Unified Diff: LayoutTests/navigatorcontentutils/is-protocol-handler-registered.html

Issue 392993005: Custom handlers should throw SecurityError exception if the URL's origin differs from the document'… (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Done Created 6 years, 3 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: LayoutTests/navigatorcontentutils/is-protocol-handler-registered.html
diff --git a/LayoutTests/navigatorcontentutils/is-protocol-handler-registered.html b/LayoutTests/navigatorcontentutils/is-protocol-handler-registered.html
deleted file mode 100644
index 0fd435a5eeda422588f1017236952b2a19112f40..0000000000000000000000000000000000000000
--- a/LayoutTests/navigatorcontentutils/is-protocol-handler-registered.html
+++ /dev/null
@@ -1,70 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-<script src="../resources/js-test.js"></script>
-</head>
-<body>
-<p>This test makes sure that navigator.isProtocolHandlerRegistered throws the proper exceptions and returns the default state of handler.</p>
-<pre id="console"></pre>
-<script>
-if (window.testRunner)
- testRunner.dumpAsText();
-
-if (window.internals)
- internals.setNavigatorContentUtilsClientMock(document);
-
-if (window.navigator.isProtocolHandlerRegistered)
- debug('PASS window.navigator.isProtocolHandlerRegistered is defined.');
-else
- debug('FAIL window.navigator.isProtocolHandlerRegistered is not defined.');
-
-var invalidUrl = "%S";
-var succeeded = false;
-try {
- window.navigator.isProtocolHandlerRegistered(scheme, invalidUrl);
- succeeded = false;
-} catch (e) {
- succeeded = true;
-}
-if (succeeded)
- debug('PASS Invalid url "' + invalidUrl + '" threw SyntaxError exception.');
-else
- debug('FAIL Invalid url "' + invalidUrl + '" allowed.');
-
-var invalid_schemes = ['mailto:', 'ssh:/', 'magnet:+', 'tel:sip'];
-invalid_schemes.forEach(function (scheme) {
- var succeeded = false;
- try {
- window.navigator.isProtocolHandlerRegistered(scheme, 'invalid scheme uri=%s');
- } catch (e) {
- succeeded = 'SecurityError' == e.name;
- errorMessage = e.message;
- }
-
- if (succeeded)
- debug('PASS Invalid scheme "' + scheme + '" falied.');
- else
- debug('FAIL Invalid scheme "' + scheme + '" allowed. Threw exception: "' + errorMessage + '".');
-});
-
-debug("\nCheck if isProtocolHandlerRegistered() works correctly. If isProtocolHandlerRegistered() returns 'new' state, it works well.");
-debug("'bitcoin' scheme will be registered, and then checks if the 'bitcoin' scheme is registered by isProtocolHandlerRegistered().\n");
-try {
- // Register 'bitcoin' scheme for testing.
- window.navigator.registerProtocolHandler('bitcoin', 'invalid scheme uri=%s', 'title');
-
- var state = window.navigator.isProtocolHandlerRegistered("bitcoin", "valid scheme %s");
- if (state == "registered")
- debug("PASS window.navigator.isProtocolHandlerRegistered returns 'registered' state. isProtocolHandlerRegistered() works correctly.");
- else if (state == "new")
- debug("FAIL window.navigator.isProtocolHandlerRegistered returns 'new' state.");
- else if (state == "declined")
- debug("FAIL window.navigator.isProtocolHandlerRegistered returns 'declined' state.");
-} catch (e) {
- debug('FAIL window.navigator.isProtocolHandlerRegistered call is failed: "' + e.message + '".');
-}
-debug("\n");
-
-</script>
-</body>
-</html>

Powered by Google App Engine
This is Rietveld 408576698