Commit 50194cc0 authored by Matt Falkenhagen's avatar Matt Falkenhagen Committed by Commit Bot

Fetch API: Remove obsolete TODO about service worker response url.

The response URL is now used without breaking the DCHECK. This function
uses |url_list_| and |response| to populate response_data->SetURLList(),
with the service worker response url as necessary, but |url_list_|
doesn't have the service worker response url.

Bug: 553535
Change-Id: Id42bdb3b62d52be6b043ef16ff4a6347d8df1fb9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2016807Reviewed-by: default avatarYutaka Hirano <yhirano@chromium.org>
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#735322}
parent b7dbfbe8
......@@ -319,9 +319,6 @@ void FetchManager::Loader::DidReceiveResponse(
const ResourceResponse& response) {
// Verify that we're dealing with the URL we expect (which could be an
// HTTPS-upgraded variant of `url_list_.back()`.
//
// TODO(horo): This check could be false when we will use the response url
// in service worker responses. (crbug.com/553535)
DCHECK(
response.CurrentRequestUrl() == url_list_.back() ||
(response.CurrentRequestUrl().ProtocolIs("https") &&
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment