New issue
Advanced search Search tips

Issue 854666 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 15
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 2
Type: Bug

Blocking:
issue 855267



Sign in to add a comment

Add mechanism to disable/defer loading of certain subresources based on resource loading hints

Project Member Reported by tbansal@chromium.org, Jun 20 2018

Issue description

We need to add a mechanism to disable and/or defer loading of certain subresources in Blink based on loading hints available in the browser process. The goal here is to speed up page loads by not loading (or lazily loading) subresources that may not be necessary.
 
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 21 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/17bbc8ffb4019beea505028a818f9ba9487c4ef1

commit 17bbc8ffb4019beea505028a818f9ba9487c4ef1
Author: Tarun Bansal <tbansal@chromium.org>
Date: Thu Jun 21 18:26:52 2018

Add Resource loading hints optimization to the list of previews

Add RESOURCE_LOADING_HINTS optimization to the list of previews.
Also, check if this optimization is enabled using base::Features,
and if so, perform rest of the checks (including optimization hints),
and add it to the list of enabled and/or committed previews.

Note that currently this only supports resource loading hints provided
via optimization hints and not via real time server API.

Change-Id: I3764c05d731046a262eaa14b0f33d5caf8132fb5
TBR: holte@chromium.org
Bug:  854666 
Reviewed-on: https://chromium-review.googlesource.com/1107158
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
Reviewed-by: Doug Arnett <dougarnett@chromium.org>
Cr-Commit-Position: refs/heads/master@{#569327}
[modify] https://crrev.com/17bbc8ffb4019beea505028a818f9ba9487c4ef1/chrome/browser/previews/previews_infobar_delegate.cc
[modify] https://crrev.com/17bbc8ffb4019beea505028a818f9ba9487c4ef1/chrome/browser/previews/previews_service.cc
[modify] https://crrev.com/17bbc8ffb4019beea505028a818f9ba9487c4ef1/components/previews/content/previews_content_util.cc
[modify] https://crrev.com/17bbc8ffb4019beea505028a818f9ba9487c4ef1/components/previews/content/previews_content_util_unittest.cc
[modify] https://crrev.com/17bbc8ffb4019beea505028a818f9ba9487c4ef1/components/previews/content/previews_io_data.cc
[modify] https://crrev.com/17bbc8ffb4019beea505028a818f9ba9487c4ef1/components/previews/content/previews_io_data_unittest.cc
[modify] https://crrev.com/17bbc8ffb4019beea505028a818f9ba9487c4ef1/components/previews/core/previews_experiments.cc
[modify] https://crrev.com/17bbc8ffb4019beea505028a818f9ba9487c4ef1/components/previews/core/previews_experiments.h
[modify] https://crrev.com/17bbc8ffb4019beea505028a818f9ba9487c4ef1/components/previews/core/previews_features.cc
[modify] https://crrev.com/17bbc8ffb4019beea505028a818f9ba9487c4ef1/components/previews/core/previews_features.h
[modify] https://crrev.com/17bbc8ffb4019beea505028a818f9ba9487c4ef1/content/public/common/previews_state.h
[modify] https://crrev.com/17bbc8ffb4019beea505028a818f9ba9487c4ef1/third_party/blink/public/platform/web_url_request.h
[modify] https://crrev.com/17bbc8ffb4019beea505028a818f9ba9487c4ef1/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/17bbc8ffb4019beea505028a818f9ba9487c4ef1/tools/metrics/histograms/histograms.xml

Blocking: 855267
Project Member

Comment 4 by bugdroid1@chromium.org, Jul 9

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d466bf9759ccfa2dd72902f5e53694302d2234eb

commit d466bf9759ccfa2dd72902f5e53694302d2234eb
Author: Tarun Bansal <tbansal@chromium.org>
Date: Mon Jul 09 22:19:44 2018

Add mojom call to send resource loading hints from browser to renderer.

The mojom call is send only when resource loading hints feature is
enabled and the connection is slow.

Cq-Include-Trybots: luci.chromium.try:linux_mojo
Change-Id: Ib57d656406db6f63200ffbf5dbbe83c356d464ca
Bug:  854666 
Reviewed-on: https://chromium-review.googlesource.com/1112925
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
Reviewed-by: Doug Arnett <dougarnett@chromium.org>
Cr-Commit-Position: refs/heads/master@{#573485}
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/chrome/browser/BUILD.gn
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/chrome/browser/chrome_content_renderer_manifest_overlay.json
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/chrome/browser/previews/previews_browsertest.cc
[add] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/chrome/browser/previews/resource_loading_hints/resource_loading_hints_browsertest.cc
[add] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/chrome/browser/previews/resource_loading_hints/resource_loading_hints_web_contents_observer.cc
[add] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/chrome/browser/previews/resource_loading_hints/resource_loading_hints_web_contents_observer.h
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/chrome/browser/ui/tab_helpers.cc
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/chrome/test/BUILD.gn
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/components/optimization_guide/proto/hints.proto
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/components/optimization_guide/test_component_creator.cc
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/components/optimization_guide/test_component_creator.h
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/components/previews/content/previews_content_util.cc
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/components/previews/content/previews_decider_impl.cc
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/testing/buildbot/filters/mojo.fyi.network_browser_tests.filter
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/third_party/blink/public/mojom/BUILD.gn
[add] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/third_party/blink/public/mojom/loader/previews_resource_loading_hints.mojom
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/third_party/blink/renderer/controller/blink_initializer.cc
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/third_party/blink/renderer/core/frame/local_frame.cc
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/third_party/blink/renderer/core/frame/local_frame.h
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/third_party/blink/renderer/core/loader/BUILD.gn
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/third_party/blink/renderer/core/loader/base_fetch_context.cc
[add] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/third_party/blink/renderer/core/loader/previews_resource_loading_hints_receiver_impl.cc
[add] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/third_party/blink/renderer/core/loader/previews_resource_loading_hints_receiver_impl.h
[modify] https://crrev.com/d466bf9759ccfa2dd72902f5e53694302d2234eb/tools/metrics/histograms/histograms.xml

Project Member

Comment 5 by bugdroid1@chromium.org, Jul 10

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d785ccc1b07d1bcf00a8645213c40774d64065b8

commit d785ccc1b07d1bcf00a8645213c40774d64065b8
Author: Joshua Pawlicki <waffles@chromium.org>
Date: Tue Jul 10 13:18:32 2018

Revert "Add mojom call to send resource loading hints from browser to renderer."

This reverts commit d466bf9759ccfa2dd72902f5e53694302d2234eb.

Reason for revert: Introduces flakiness in ResourceLoadingHintsBrowserTest.ResourceLoadingHintsHttpsWhitelistedRedirectToHttps

Looks like:
"Failure\nExpected: (nullptr) != (histogram), actual: (nullptr) vs NULL\nHistogram \"ResourceLoadingHints.CountBlockedSubresourcePatterns\" does not exist."

Original change's description:
> Add mojom call to send resource loading hints from browser to renderer.
> 
> The mojom call is send only when resource loading hints feature is
> enabled and the connection is slow.
> 
> Cq-Include-Trybots: luci.chromium.try:linux_mojo
> Change-Id: Ib57d656406db6f63200ffbf5dbbe83c356d464ca
> Bug:  854666 
> Reviewed-on: https://chromium-review.googlesource.com/1112925
> Commit-Queue: Tarun Bansal <tbansal@chromium.org>
> Reviewed-by: Daniel Cheng <dcheng@chromium.org>
> Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
> Reviewed-by: Peter Kasting <pkasting@chromium.org>
> Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
> Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
> Reviewed-by: Doug Arnett <dougarnett@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#573485}

TBR=pkasting@chromium.org,dcheng@chromium.org,kinuko@chromium.org,rkaplow@chromium.org,tbansal@chromium.org,dougarnett@chromium.org,ryansturm@chromium.org

Change-Id: I7481f8505d9d0c8d08d3eb91e577fbcc9fb0b4d3
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug:  854666 
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Reviewed-on: https://chromium-review.googlesource.com/1131294
Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
Commit-Queue: Joshua Pawlicki <waffles@chromium.org>
Cr-Commit-Position: refs/heads/master@{#573703}
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/chrome/browser/BUILD.gn
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/chrome/browser/chrome_content_renderer_manifest_overlay.json
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/chrome/browser/previews/previews_browsertest.cc
[delete] https://crrev.com/7c1b7a0d327d462715047240195ab44ef0fc2829/chrome/browser/previews/resource_loading_hints/resource_loading_hints_browsertest.cc
[delete] https://crrev.com/7c1b7a0d327d462715047240195ab44ef0fc2829/chrome/browser/previews/resource_loading_hints/resource_loading_hints_web_contents_observer.cc
[delete] https://crrev.com/7c1b7a0d327d462715047240195ab44ef0fc2829/chrome/browser/previews/resource_loading_hints/resource_loading_hints_web_contents_observer.h
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/chrome/browser/ui/tab_helpers.cc
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/chrome/test/BUILD.gn
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/components/optimization_guide/proto/hints.proto
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/components/optimization_guide/test_component_creator.cc
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/components/optimization_guide/test_component_creator.h
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/components/previews/content/previews_content_util.cc
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/components/previews/content/previews_decider_impl.cc
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/testing/buildbot/filters/mojo.fyi.network_browser_tests.filter
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/third_party/blink/public/mojom/BUILD.gn
[delete] https://crrev.com/7c1b7a0d327d462715047240195ab44ef0fc2829/third_party/blink/public/mojom/loader/previews_resource_loading_hints.mojom
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/third_party/blink/renderer/controller/blink_initializer.cc
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/third_party/blink/renderer/core/frame/local_frame.cc
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/third_party/blink/renderer/core/frame/local_frame.h
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/third_party/blink/renderer/core/loader/BUILD.gn
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/third_party/blink/renderer/core/loader/base_fetch_context.cc
[delete] https://crrev.com/7c1b7a0d327d462715047240195ab44ef0fc2829/third_party/blink/renderer/core/loader/previews_resource_loading_hints_receiver_impl.cc
[delete] https://crrev.com/7c1b7a0d327d462715047240195ab44ef0fc2829/third_party/blink/renderer/core/loader/previews_resource_loading_hints_receiver_impl.h
[modify] https://crrev.com/d785ccc1b07d1bcf00a8645213c40774d64065b8/tools/metrics/histograms/histograms.xml

Project Member

Comment 6 by bugdroid1@chromium.org, Jul 12

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/dc97029f96fedad41b8600011d6c155810e50fe6

commit dc97029f96fedad41b8600011d6c155810e50fe6
Author: Tarun Bansal <tbansal@chromium.org>
Date: Thu Jul 12 00:02:58 2018

Reland of: Add mojom call to send resource loading hints from browser to renderer.

The mojom call is send only when resource loading hints feature is
enabled and the connection is slow.

This is a reland of the CL that got reverted due to flaky
browsertest. This reland is quite similar to the reverted CL,
except for some changes in the browsertest.

PS#1 is the original CL that got reverted.

The fix is to retry fetching of histograms from the renderers.
Note that previews_browsertest are currently enabled only on Android.
So, it's possible that the same flakiness bug is affecting the
browser tests on Linux in this CL.

Bug:  854666 
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Change-Id: I753e488bef7318e43530a5594a94b24bd09295bb
TBR: rkaplow@chromium.org, dcheng@chromium.org, pkasting@chromium.org, kinuko@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/1131918
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#574419}
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/chrome/browser/BUILD.gn
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/chrome/browser/chrome_content_renderer_manifest_overlay.json
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/chrome/browser/previews/previews_browsertest.cc
[add] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/chrome/browser/previews/resource_loading_hints/resource_loading_hints_browsertest.cc
[add] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/chrome/browser/previews/resource_loading_hints/resource_loading_hints_web_contents_observer.cc
[add] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/chrome/browser/previews/resource_loading_hints/resource_loading_hints_web_contents_observer.h
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/chrome/browser/ui/tab_helpers.cc
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/chrome/test/BUILD.gn
[add] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/chrome/test/data/previews/resource_loading_hints.html
[add] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/chrome/test/data/previews/resource_loading_hints_with_no_transform_header.html
[add] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/chrome/test/data/previews/resource_loading_hints_with_no_transform_header.html.mock-http-headers
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/components/optimization_guide/proto/hints.proto
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/components/optimization_guide/test_component_creator.cc
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/components/optimization_guide/test_component_creator.h
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/components/previews/content/previews_content_util.cc
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/components/previews/content/previews_decider_impl.cc
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/testing/buildbot/filters/mojo.fyi.network_browser_tests.filter
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/third_party/blink/public/mojom/BUILD.gn
[add] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/third_party/blink/public/mojom/loader/previews_resource_loading_hints.mojom
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/third_party/blink/renderer/controller/blink_initializer.cc
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/third_party/blink/renderer/core/frame/local_frame.cc
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/third_party/blink/renderer/core/frame/local_frame.h
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/third_party/blink/renderer/core/loader/BUILD.gn
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/third_party/blink/renderer/core/loader/base_fetch_context.cc
[add] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/third_party/blink/renderer/core/loader/previews_resource_loading_hints_receiver_impl.cc
[add] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/third_party/blink/renderer/core/loader/previews_resource_loading_hints_receiver_impl.h
[modify] https://crrev.com/dc97029f96fedad41b8600011d6c155810e50fe6/tools/metrics/histograms/histograms.xml

Project Member

Comment 7 by bugdroid1@chromium.org, Jul 20

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/833dac7668fd47051c7955e2f23eb1590eec0556

commit 833dac7668fd47051c7955e2f23eb1590eec0556
Author: Tarun Bansal <tbansal@chromium.org>
Date: Fri Jul 20 16:37:13 2018

Disable loading of subresources based on previews resource loading hints

Currently, the browser provides the renderer with previews resource loading
hints when the relevant experiment is enabled.

This CL adds logic to disable loading of subresources when their URL
matches the provided resource loading hints.

The CL does not provide appropriate plumbing to dev tools when a
resource load is blocked. That plumbing would be completed in a
subsequent CL.

Change-Id: I7df4db03cfa39e569c7641a1bd72406038e24b21
Bug:  856247 , 854666 
Reviewed-on: https://chromium-review.googlesource.com/1139903
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Doug Arnett <dougarnett@chromium.org>
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#576898}
[modify] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/chrome/browser/previews/resource_loading_hints/resource_loading_hints_browsertest.cc
[modify] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/chrome/test/data/previews/resource_loading_hints.html
[modify] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/chrome/test/data/previews/resource_loading_hints_with_no_transform_header.html
[modify] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/third_party/blink/renderer/core/BUILD.gn
[modify] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/third_party/blink/renderer/core/frame/local_frame.cc
[modify] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/third_party/blink/renderer/core/loader/BUILD.gn
[modify] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/third_party/blink/renderer/core/loader/base_fetch_context.cc
[modify] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/third_party/blink/renderer/core/loader/base_fetch_context.h
[modify] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/third_party/blink/renderer/core/loader/base_fetch_context_test.cc
[modify] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/third_party/blink/renderer/core/loader/document_loader.cc
[modify] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/third_party/blink/renderer/core/loader/document_loader.h
[modify] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/third_party/blink/renderer/core/loader/frame_fetch_context.cc
[modify] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/third_party/blink/renderer/core/loader/frame_fetch_context.h
[add] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/third_party/blink/renderer/core/loader/previews_resource_loading_hints.cc
[add] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/third_party/blink/renderer/core/loader/previews_resource_loading_hints.h
[modify] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/third_party/blink/renderer/core/loader/previews_resource_loading_hints_receiver_impl.cc
[modify] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/third_party/blink/renderer/core/loader/previews_resource_loading_hints_receiver_impl.h
[add] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/third_party/blink/renderer/core/loader/previews_resource_loading_hints_test.cc
[modify] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/third_party/blink/renderer/core/loader/worker_fetch_context.cc
[modify] https://crrev.com/833dac7668fd47051c7955e2f23eb1590eec0556/third_party/blink/renderer/core/loader/worker_fetch_context.h

Status: Fixed (was: Started)
The "disable" part is done. We have the mojo pipeline that sends the hints from browser to renderer, and appropriate renderer code to block loading of matching subresources. Lets, open separate bugs for "defer".

Sign in to add a comment