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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/cachestorage/serviceworker/credentials.html

Issue 2651943002: Block subresource requests whose URLs include credentials. (Closed)
Patch Set: Test. Created 3 years, 9 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/cachestorage/serviceworker/credentials.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/cachestorage/serviceworker/credentials.html b/third_party/WebKit/LayoutTests/http/tests/cachestorage/serviceworker/credentials.html
index 1bceb40e656e80c1b03732aeb709fad9a8f5b1c4..0bd155673976f133aecd809813d595608768b783 100644
--- a/third_party/WebKit/LayoutTests/http/tests/cachestorage/serviceworker/credentials.html
+++ b/third_party/WebKit/LayoutTests/http/tests/cachestorage/serviceworker/credentials.html
@@ -34,12 +34,12 @@ promise_test(function(t) {
});
})
.then(function(data) {
- assert_equals(data.length, 3, 'three entries should be present');
- assert_equals(data.filter(function(url) { return /@/.test(url); }).length, 2,
+ assert_equals(data.length, 1, 'three entries should be present');
+ assert_equals(data.filter(function(url) { return /@/.test(url); }).length, 0,
'two entries should contain credentials');
- assert_true(data.some(function(url) { return /aa:bb@/.test(url); }),
+ assert_false(data.some(function(url) { return /aa:bb@/.test(url); }),
'entry with credentials aa:bb should be present');
- assert_true(data.some(function(url) { return /cc:dd@/.test(url); }),
+ assert_false(data.some(function(url) { return /cc:dd@/.test(url); }),
'entry with credentials cc:dd should be present');
});
});

Powered by Google App Engine
This is Rietveld 408576698