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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium/service-worker-mixed-response.html

Issue 1993083002: The cross-origin checks in the multibuffer code are not sufficient, as they only trigger when a red… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2704
Patch Set: Created 4 years, 7 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
« no previous file with comments | « media/blink/url_index.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium/service-worker-mixed-response.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium/service-worker-mixed-response.html b/third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium/service-worker-mixed-response.html
index bd0e03361453c10e65747e98828dd6ff99804d55..c0772b4fa5b2d265f8c5417864fbcc7c5bfbd01d 100644
--- a/third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium/service-worker-mixed-response.html
+++ b/third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium/service-worker-mixed-response.html
@@ -68,6 +68,8 @@ function create_success_audio_test(frame, url) {
// URL of it. And when it receives the succeeding response, it checks the origin
// of the new response. If the origin is not same as the origin of the first
// response, the response must be treated as an error.
+
+var request_num = 1;
function audio_test(frame, original, first_byte, second_byte, expect_success) {
var url;
var HOST_INFO = get_host_info();
@@ -79,6 +81,8 @@ function audio_test(frame, original, first_byte, second_byte, expect_success) {
url = HOST_INFO['HTTP_REMOTE_ORIGIN'] + AUDIO_PATH;
}
url += '&SW_FIRST=' + first_byte + '&SW_SECOND=' + second_byte;
+ url += '&prevent_cache=' + request_num;
+ request_num += 1;
if (expect_success) {
return create_success_audio_test(frame, url);
} else {
« no previous file with comments | « media/blink/url_index.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698