Add support for blob urls for subresources.

Add support for blob urls for subresources.

BUG=715677
Review-Url: https://codereview.chromium.org/2925903004
Cr-Commit-Position: refs/heads/master@{#477539}
Committed: https://chromium.googlesource.com/chromium/src/+/188f19f9e26e879ac7aab7ac4bc54ba8cc57eb91

[jam, 2017-06-06 20:41:26]

The CQ bit was checked by jam@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-06 20:41:48]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jam, 2017-06-06 21:43:19]

jam@chromium.org changed reviewers:
+ dcheng@chromium.org, rockot@chromium.org

[jam, 2017-06-06 21:43:20]

Daniel: security review of mojom/manifest (and anything else you want)
Ken: overall

[Ken Rockot(use gerrit already), 2017-06-06 21:49:58]

lgtm

https://codereview.chromium.org/2925903004/diff/1/content/common/renderer_hos...
File content/common/renderer_host.mojom (right):

https://codereview.chromium.org/2925903004/diff/1/content/common/renderer_hos...
content/common/renderer_host.mojom:12: // Returns a URLLoaderFactory for the
blob scheme.
super nit: maybe s/Returns/Requests/

[commit-bot: I haz the power, 2017-06-06 22:02:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-06 22:02:23]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[jam, 2017-06-06 22:14:38]

The CQ bit was checked by jam@chromium.org to run a CQ dry run

[jam, 2017-06-06 22:14:39]

https://codereview.chromium.org/2925903004/diff/1/content/common/renderer_hos...
File content/common/renderer_host.mojom (right):

https://codereview.chromium.org/2925903004/diff/1/content/common/renderer_hos...
content/common/renderer_host.mojom:12: // Returns a URLLoaderFactory for the
blob scheme.
On 2017/06/06 21:49:58, Ken Rockot(use gerrit already) wrote:
> super nit: maybe s/Returns/Requests/

Done.

[commit-bot: I haz the power, 2017-06-06 22:15:00]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[dcheng, 2017-06-06 23:05:47]

mojo lgtm

https://codereview.chromium.org/2925903004/diff/20001/content/renderer/render...
File content/renderer/render_thread_impl.cc (right):

https://codereview.chromium.org/2925903004/diff/20001/content/renderer/render...
content/renderer/render_thread_impl.cc:1088: if (!renderer_host_.get()) {
Nit: no .get(), here and below

[commit-bot: I haz the power, 2017-06-07 00:02:41]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-07 00:02:42]

Dry run: This issue passed the CQ dry run.

[jam, 2017-06-07 00:03:27]

The CQ bit was checked by jam@chromium.org

[jam, 2017-06-07 00:03:28]

The patchset sent to the CQ was uploaded after l-g-t-m from rockot@chromium.org,
dcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/2925903004/#ps40001
(title: "review comment")

[commit-bot: I haz the power, 2017-06-07 00:03:45]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-07 01:11:20]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-07 01:11:22]

Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[jam, 2017-06-07 03:32:24]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2017-06-07 03:32:36]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jam, 2017-06-07 03:32:41]

https://codereview.chromium.org/2925903004/diff/20001/content/renderer/render...
File content/renderer/render_thread_impl.cc (right):

https://codereview.chromium.org/2925903004/diff/20001/content/renderer/render...
content/renderer/render_thread_impl.cc:1088: if (!renderer_host_.get()) {
On 2017/06/06 23:05:46, dcheng wrote:
> Nit: no .get(), here and below

Done.

[commit-bot: I haz the power, 2017-06-07 03:56:27]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1496806344016190,
"parent_rev": "bfa7caded079e8dc4b8414cadf2264fe978b5059", "commit_rev":
"188f19f9e26e879ac7aab7ac4bc54ba8cc57eb91"}

[commit-bot: I haz the power, 2017-06-07 03:56:35]

Description was changed from

==========
Add support for blob urls for subresources.

BUG=715677
==========

to

==========
Add support for blob urls for subresources.

BUG=715677

Review-Url: https://codereview.chromium.org/2925903004
Cr-Commit-Position: refs/heads/master@{#477539}
Committed:
https://chromium.googlesource.com/chromium/src/+/188f19f9e26e879ac7aab7ac4bc5...
==========

[commit-bot: I haz the power, 2017-06-07 03:56:36]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/188f19f9e26e879ac7aab7ac4bc5...

[kinuko, 2017-06-07 04:58:34]

kinuko@chromium.org changed reviewers:
+ kinuko@chromium.org

[kinuko, 2017-06-07 04:58:35]

https://codereview.chromium.org/2925903004/diff/40001/content/renderer/render...
File content/renderer/render_frame_impl.cc (right):

https://codereview.chromium.org/2925903004/diff/40001/content/renderer/render...
content/renderer/render_frame_impl.cc:4313: // TODO: generalize how non-network
schemes are sent to the renderer and used.
Filed a bug- crbug.com/730338