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

Issue 27536014: Teach mojo_shell how to load data URLs (Closed)

Created:
7 years, 2 months ago by abarth-chromium
Modified:
7 years, 2 months ago
CC:
chromium-reviews, Aaron Boodman, darin (slow to review), ben+mojo_chromium.org, viettrungluu+watch_chromium.org
Visibility:
Public.

Description

Teach mojo_shell how to load data URLs This CL contains enough integration with net to load data URLs. A future CL will add the ability to load http and https URLs. Data URLs are easier because they're entirely in-memory. R=aa@chromium.org

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+203 lines, -0 lines) Patch
A + mojo/loader/DEPS View 1 chunk +1 line, -0 lines 0 comments Download
A mojo/loader/loader.h View 1 chunk +39 lines, -0 lines 0 comments Download
A mojo/loader/loader.cc View 1 chunk +66 lines, -0 lines 0 comments Download
A mojo/loader/url_request_context_getter.h View 1 chunk +35 lines, -0 lines 0 comments Download
A mojo/loader/url_request_context_getter.cc View 1 chunk +57 lines, -0 lines 0 comments Download
M mojo/mojo.gyp View 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
abarth-chromium
7 years, 2 months ago (2013-10-17 23:33:51 UTC) #1
abarth-chromium
Let me know where you'd like this code to go. I put it in a ...
7 years, 2 months ago (2013-10-17 23:34:35 UTC) #2
abarth-chromium
+darin in case he has opinions on where this code should live.
7 years, 2 months ago (2013-10-17 23:55:22 UTC) #3
abarth-chromium
7 years, 2 months ago (2013-10-18 06:50:52 UTC) #4
Closing this issue.  I'm going to roll this all into
https://codereview.chromium.org/27943002/.

Powered by Google App Engine
This is Rietveld 408576698