Issue 771593006: Properly handle defers loading in WebURLLoaderImpl for data uris

Issue 771593006: Properly handle defers loading in WebURLLoaderImpl for data uris (Closed)

Created:
6 years ago by João Eiras

Modified:
6 years ago

Reviewers:
Ken Russell (switch to Gerrit), jochen (gone - plz use gerrit), davidben

CC:
chromium-reviews, darin-cc_chromium.org, jam

Base URL:
https://chromium.googlesource.com/chromium/src.git@master

Project:
chromium

Visibility:
Public.

Properly handle defers loading in WebURLLoaderImpl for data uris BUG=418116 Committed: https://crrev.com/645808fdc15f83f8fa98cc9d78f9b64289fa359b Cr-Commit-Position: refs/heads/master@{#307266}

Created: 6 years ago

		Unified diffs	Side-by-side diffs	Stats (+59 lines, -2 lines)
	M	content/child/web_url_loader_impl.cc	View	5 chunks	+20 lines, -1 line	0 comments
	M	content/child/web_url_loader_impl_unittest.cc	View	2 chunks	+39 lines, -1 line	0 comments

Total messages: 9 (3 generated)

Ken Russell (switch to Gerrit)

LGTM, but I'm not an OWNER. Suggest you CC: more OWNERS.

6 years ago (2014-12-03 23:37:29 UTC) #3

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/771593006/20001

6 years ago (2014-12-08 16:16:27 UTC) #7

commit-bot: I haz the power

6 years ago (2014-12-08 17:07:41 UTC) #9

Message was sent while issue was closed.

Patchset 2 (id:??) landed as
https://crrev.com/645808fdc15f83f8fa98cc9d78f9b64289fa359b
Cr-Commit-Position: refs/heads/master@{#307266}