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

Issue 1283163003: [Fetch][WIP] Support blob:, about:, and data: URLs. (Closed)

Created:
5 years, 4 months ago by hiroshige
Modified:
5 years, 2 months ago
Reviewers:
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Target Ref:
refs/heads/master
Project:
blink
Visibility:
Public.

Description

[Fetch] Support blob:, about:, and data: URLs. BUG=

Patch Set 1 #

Patch Set 2 : Base revision with all tests / schemes. #

Patch Set 3 : Cleanup. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+245 lines, -34 lines) Patch
M LayoutTests/http/tests/fetch/resources/thorough-util.js View 1 2 2 chunks +6 lines, -1 line 0 comments Download
M LayoutTests/http/tests/fetch/script-tests/fetch.js View 1 2 1 chunk +71 lines, -0 lines 0 comments Download
A LayoutTests/http/tests/fetch/script-tests/thorough/scheme-blob.js View 1 2 1 chunk +24 lines, -0 lines 0 comments Download
A LayoutTests/http/tests/fetch/script-tests/thorough/scheme-data.js View 1 1 chunk +46 lines, -0 lines 0 comments Download
A + LayoutTests/http/tests/fetch/serviceworker-proxied/thorough/scheme-blob.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + LayoutTests/http/tests/fetch/serviceworker-proxied/thorough/scheme-blob-base-https-other-https.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + LayoutTests/http/tests/fetch/serviceworker-proxied/thorough/scheme-blob-other-https.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + LayoutTests/http/tests/fetch/serviceworker-proxied/thorough/scheme-data.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + LayoutTests/http/tests/fetch/serviceworker-proxied/thorough/scheme-data-base-https-other-https.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + LayoutTests/http/tests/fetch/serviceworker-proxied/thorough/scheme-data-other-https.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + LayoutTests/http/tests/fetch/serviceworker/thorough/scheme-blob.html View 1 2 1 chunk +2 lines, -2 lines 0 comments Download
A + LayoutTests/http/tests/fetch/serviceworker/thorough/scheme-blob-base-https-other-https.html View 1 2 1 chunk +2 lines, -2 lines 0 comments Download
A + LayoutTests/http/tests/fetch/serviceworker/thorough/scheme-blob-other-https.html View 1 2 1 chunk +2 lines, -2 lines 0 comments Download
A + LayoutTests/http/tests/fetch/serviceworker/thorough/scheme-data.html View 1 2 1 chunk +2 lines, -2 lines 0 comments Download
A + LayoutTests/http/tests/fetch/serviceworker/thorough/scheme-data-base-https-other-https.html View 1 2 1 chunk +2 lines, -2 lines 0 comments Download
A + LayoutTests/http/tests/fetch/serviceworker/thorough/scheme-data-other-https.html View 1 2 1 chunk +2 lines, -2 lines 0 comments Download
A + LayoutTests/http/tests/fetch/window/thorough/scheme-blob.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + LayoutTests/http/tests/fetch/window/thorough/scheme-blob-base-https-other-https.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + LayoutTests/http/tests/fetch/window/thorough/scheme-blob-other-https.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + LayoutTests/http/tests/fetch/window/thorough/scheme-data.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + LayoutTests/http/tests/fetch/window/thorough/scheme-data-base-https-other-https.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + LayoutTests/http/tests/fetch/window/thorough/scheme-data-other-https.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + LayoutTests/http/tests/fetch/workers/thorough/scheme-blob.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + LayoutTests/http/tests/fetch/workers/thorough/scheme-blob-base-https-other-https.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + LayoutTests/http/tests/fetch/workers/thorough/scheme-blob-other-https.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + LayoutTests/http/tests/fetch/workers/thorough/scheme-data.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + LayoutTests/http/tests/fetch/workers/thorough/scheme-data-base-https-other-https.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + LayoutTests/http/tests/fetch/workers/thorough/scheme-data-other-https.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/fetch/FetchManager.cpp View 1 2 5 chunks +68 lines, -3 lines 0 comments Download

Messages

Total messages: 4 (2 generated)
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1283163003/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1283163003/20001
5 years, 4 months ago (2015-08-17 09:18:30 UTC) #2
commit-bot: I haz the power
5 years, 4 months ago (2015-08-17 11:07:35 UTC) #4
Dry run: This issue passed the CQ dry run.

Powered by Google App Engine
This is Rietveld 408576698