• Matt Menke's avatar
    Revert "Reland "Always send subresource ULF on navigation"" · f4098473
    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: default avatarMatt Menke <mmenke@chromium.org>
    Commit-Queue: Matt Menke <mmenke@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#512779}
    f4098473
frame.mojom 6.1 KB