New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 602137 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2016
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 594159



Sign in to add a comment

Add about:flags support for doc.written script blocking

Project Member Reported by bmcquade@chromium.org, Apr 11 2016

Issue description

Several people have requested that we add about:flags support so they can try out the experimental feature to block the load of doc.written scripts.
 
Blocking: 594159
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 12 2016

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

commit ed75dc7053dc072bdd0e72f64a869d59ec587f4b
Author: bmcquade <bmcquade@chromium.org>
Date: Tue Apr 12 19:41:23 2016

Add about:flags support for doc.write script blocking.

This patch introduces an additional blink setting which allows us to enable
blocking only on slow (2G) connections, or on all connections.

This is consistent with the 2G web font intervention, which has a feature
for enabling on 2G, and an additional feature to force the intervention on
in all connection types.

We then add support for enabling the setting that enables blocking on all
connection types from about:flags.

This change also removes about:flags support from the now discontinued low
priority iframe experiment, since it's nontrivial to add support for
multiple blink settings in about:flags.

BUG= 602137 

Review URL: https://codereview.chromium.org/1868413002

Cr-Commit-Position: refs/heads/master@{#386769}

[modify] https://crrev.com/ed75dc7053dc072bdd0e72f64a869d59ec587f4b/chrome/app/generated_resources.grd
[modify] https://crrev.com/ed75dc7053dc072bdd0e72f64a869d59ec587f4b/chrome/browser/about_flags.cc
[modify] https://crrev.com/ed75dc7053dc072bdd0e72f64a869d59ec587f4b/third_party/WebKit/LayoutTests/http/tests/loading/doc-write-async-third-party-script.html
[add] https://crrev.com/ed75dc7053dc072bdd0e72f64a869d59ec587f4b/third_party/WebKit/LayoutTests/http/tests/loading/doc-write-sync-third-party-script-block-all-conn-types-expected.txt
[add] https://crrev.com/ed75dc7053dc072bdd0e72f64a869d59ec587f4b/third_party/WebKit/LayoutTests/http/tests/loading/doc-write-sync-third-party-script-block-all-conn-types.html
[modify] https://crrev.com/ed75dc7053dc072bdd0e72f64a869d59ec587f4b/third_party/WebKit/LayoutTests/http/tests/loading/doc-write-sync-third-party-script-block.html
[modify] https://crrev.com/ed75dc7053dc072bdd0e72f64a869d59ec587f4b/third_party/WebKit/LayoutTests/http/tests/loading/doc-write-sync-third-party-script-conn-type.html
[modify] https://crrev.com/ed75dc7053dc072bdd0e72f64a869d59ec587f4b/third_party/WebKit/Source/core/dom/Document.h
[modify] https://crrev.com/ed75dc7053dc072bdd0e72f64a869d59ec587f4b/third_party/WebKit/Source/core/frame/Settings.in
[modify] https://crrev.com/ed75dc7053dc072bdd0e72f64a869d59ec587f4b/third_party/WebKit/Source/core/loader/FrameFetchContext.cpp

Status: Fixed (was: Assigned)
Labels: Merge-Request-51

Comment 5 by tin...@google.com, Apr 13 2016

Labels: -Merge-Request-51 Merge-Approved-51 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M51 (branch: 2704)
Project Member

Comment 6 by bugdroid1@chromium.org, Apr 13 2016

Labels: -merge-approved-51 merge-merged-2704
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/afcacf06e0bf288ec36b567c454b52764b681fed

commit afcacf06e0bf288ec36b567c454b52764b681fed
Author: Josh Karlin <jkarlin@chromium.org>
Date: Wed Apr 13 19:14:59 2016

This is a merge into M51 branch 2704.

Add about:flags support for doc.write script blocking.

This patch introduces an additional blink setting which allows us to enable
blocking only on slow (2G) connections, or on all connections.

This is consistent with the 2G web font intervention, which has a feature
for enabling on 2G, and an additional feature to force the intervention on
in all connection types.

We then add support for enabling the setting that enables blocking on all
connection types from about:flags.

This change also removes about:flags support from the now discontinued low
priority iframe experiment, since it's nontrivial to add support for
multiple blink settings in about:flags.

BUG= 602137 
TBR=japhet@chromium.org

Review URL: https://codereview.chromium.org/1868413002

Cr-Commit-Position: refs/heads/master@{#386769}
(cherry picked from commit ed75dc7053dc072bdd0e72f64a869d59ec587f4b)

Review URL: https://codereview.chromium.org/1890573002 .

Cr-Commit-Position: refs/branch-heads/2704@{#33}
Cr-Branched-From: 6e53600def8f60d8c632fadc70d7c1939ccea347-refs/heads/master@{#386251}

[modify] https://crrev.com/afcacf06e0bf288ec36b567c454b52764b681fed/chrome/app/generated_resources.grd
[modify] https://crrev.com/afcacf06e0bf288ec36b567c454b52764b681fed/chrome/browser/about_flags.cc
[modify] https://crrev.com/afcacf06e0bf288ec36b567c454b52764b681fed/third_party/WebKit/LayoutTests/http/tests/loading/doc-write-async-third-party-script.html
[add] https://crrev.com/afcacf06e0bf288ec36b567c454b52764b681fed/third_party/WebKit/LayoutTests/http/tests/loading/doc-write-sync-third-party-script-block-all-conn-types-expected.txt
[add] https://crrev.com/afcacf06e0bf288ec36b567c454b52764b681fed/third_party/WebKit/LayoutTests/http/tests/loading/doc-write-sync-third-party-script-block-all-conn-types.html
[modify] https://crrev.com/afcacf06e0bf288ec36b567c454b52764b681fed/third_party/WebKit/LayoutTests/http/tests/loading/doc-write-sync-third-party-script-block.html
[modify] https://crrev.com/afcacf06e0bf288ec36b567c454b52764b681fed/third_party/WebKit/LayoutTests/http/tests/loading/doc-write-sync-third-party-script-conn-type.html
[modify] https://crrev.com/afcacf06e0bf288ec36b567c454b52764b681fed/third_party/WebKit/Source/core/dom/Document.h
[modify] https://crrev.com/afcacf06e0bf288ec36b567c454b52764b681fed/third_party/WebKit/Source/core/frame/Settings.in
[modify] https://crrev.com/afcacf06e0bf288ec36b567c454b52764b681fed/third_party/WebKit/Source/core/loader/FrameFetchContext.cpp

Sign in to add a comment