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

Unified Diff: LayoutTests/http/tests/serviceworker/registration.html

Issue 1260003003: Move Service Worker %2f validation logic from browser into Blink (2) (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 years, 5 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/http/tests/serviceworker/registration.html
diff --git a/LayoutTests/http/tests/serviceworker/registration.html b/LayoutTests/http/tests/serviceworker/registration.html
index c5ceace35b0b774d0325178d088a73cd5d29c32c..5a56e875f0456c3267a9deae329dfef30367192a 100644
--- a/LayoutTests/http/tests/serviceworker/registration.html
+++ b/LayoutTests/http/tests/serviceworker/registration.html
@@ -228,7 +228,7 @@ promise_test(function(t) {
var scope = 'resources/scope/encoded-slash-in-script-url';
return assert_promise_rejects(
navigator.serviceWorker.register(script, {scope: scope}),
- 'SecurityError',
+ new TypeError,
'URL-encoded slash in the script URL should be rejected.');
}, 'Script URL including URL-encoded slash');
@@ -237,7 +237,7 @@ promise_test(function(t) {
var scope = 'resources/scope%2fencoded-slash-in-scope';
return assert_promise_rejects(
navigator.serviceWorker.register(script, {scope: scope}),
- 'SecurityError',
+ new TypeError,
'URL-encoded slash in the scope should be rejected.');
}, 'Scope including URL-encoded slash');
@@ -246,7 +246,7 @@ promise_test(function(t) {
var scope = 'resources/scope/encoded-slash-in-script-url';
return assert_promise_rejects(
navigator.serviceWorker.register(script, {scope: scope}),
- 'SecurityError',
+ new TypeError,
'URL-encoded backslash in the script URL should be rejected.');
}, 'Script URL including URL-encoded backslash');
@@ -255,7 +255,7 @@ promise_test(function(t) {
var scope = 'resources/scope%5cencoded-slash-in-scope';
return assert_promise_rejects(
navigator.serviceWorker.register(script, {scope: scope}),
- 'SecurityError',
+ new TypeError,
'URL-encoded backslash in the scope should be rejected.');
}, 'Scope including URL-encoded backslash');
nhiroki 2015/07/28 07:04:36 It'd be good to have tests for capital letter case

Powered by Google App Engine
This is Rietveld 408576698