-
Matt Menke authored
This reverts commit e60ebbd1. Reason for revert: Caused ~60 browser_tests and content_browsertests to fail with the network service enabled. Original change's description: > Reland "Always send subresource ULF on navigation" > > This is a reland of bba21f27 > Original change's description: > > Always send subresource ULF on navigation > > > > This changes the Network Service-enabled browser-side navigation path > > to always send a default subresource URLLoaderFactory to the render > > frame on navigation commit. > > > > In addition to sending a default ULF, we also send one for blobs. > > Ultimately the idea here is to have the browser, at navigation time, > > send a frame everything it will need to request any accessible > > subresources, making it easier to reason about what subresource > > capabilities any given frame has. > > > > This set of capabilities is encapsulated by a new > > URLLoaderFactoryBundle type, which can be serialized over mojom and is > > now passed in new-document navigation commit messages. > > > > Follow-up usage of URLLoaderFactoryBundle will support loading things > > like file:// and chrome-extension:// subresources when appropriate. > > > > Note that this does not yet change how workers acquire ULFs for their > > own resource requests. > > > > Bug: 721414,759230 > > Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation > > Change-Id: I47b0897c7c012292633d5f08c874a18cc0fb1dcb > > Reviewed-on: https://chromium-review.googlesource.com/740904 > > Commit-Queue: Ken Rockot <rockot@chromium.org> > > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > > Reviewed-by: John Abd-El-Malek <jam@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#512459} > > Bug: 721414, 759230 > Change-Id: Ie13bcd97b7a5af8dbe4263a70239ba124205da15 > Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation > > TBR=dcheng@chromium.org > > Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation > Change-Id: Ie13bcd97b7a5af8dbe4263a70239ba124205da15 > Reviewed-on: https://chromium-review.googlesource.com/744222 > Commit-Queue: Ken Rockot <rockot@chromium.org> > Reviewed-by: Ken Rockot <rockot@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Reviewed-by: John Abd-El-Malek <jam@chromium.org> > Cr-Commit-Position: refs/heads/master@{#512706} TBR=dcheng@chromium.org,jam@chromium.org,rockot@chromium.org Change-Id: Iee7d5a63f105ac4c15d8320668a33d66972820b1 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 721414, 759230 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation Reviewed-on: https://chromium-review.googlesource.com/746302Reviewed-by:
Matt Menke <mmenke@chromium.org> Commit-Queue: Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#512779}
f4098473