Disable Feature-Policy HTTP header for Chrome M60 |
||||||
Issue descriptionThe syntax of the "Feature-Policy" HTTP header is under discussion at the moment, and may change radically, in a backwards-incompatible way. Given that there is essentially no usage of the header, it seems prudent to disable it for the M60 release, and reinstate it afterwards, once the spec has stabilized.
,
Jul 4 2017
,
Jul 4 2017
This bug requires manual review: M60 has already been promoted to the beta branch, so this requires manual review Please contact the milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 4 2017
,
Jul 4 2017
Fixed landed on trunk, and tested in Canary; still requires merge to M60 branch
,
Jul 7 2017
Approving merge to M60.
,
Jul 7 2017
Thanks!
,
Jul 7 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/360e15ec26cec7d5823421707839e44958678c8b commit 360e15ec26cec7d5823421707839e44958678c8b Author: Ian Clelland <iclelland@google.com> Date: Fri Jul 07 15:31:33 2017 Disable Feature-Policy HTTP Header This is a temporary measure while the header syntax is being debated, to ensure that a format which may not be the eventual standard doesn't become relied upon by sites, and accidentally become a permanent part of the web platform. BUG= 737643 TBR=iclelland@chromium.org (cherry picked from commit b78206e4c97d23230ccfd819885e8001a8bdc795) Review-Url: https://codereview.chromium.org/2936293002 Cr-Original-Commit-Position: refs/heads/master@{#483731} Change-Id: I5f1ac7446461ddcf1f53829ff7f87e804a9ce961 Reviewed-on: https://chromium-review.googlesource.com/563438 Reviewed-by: Ian Clelland <iclelland@chromium.org> Cr-Commit-Position: refs/branch-heads/3112@{#540} Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897} [modify] https://crrev.com/360e15ec26cec7d5823421707839e44958678c8b/third_party/WebKit/LayoutTests/TestExpectations [modify] https://crrev.com/360e15ec26cec7d5823421707839e44958678c8b/third_party/WebKit/Source/core/loader/DocumentLoader.cpp
,
Jul 7 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6ec7521db9dd9304049040de049f3c6819afc8b1 commit 6ec7521db9dd9304049040de049f3c6819afc8b1 Author: Ian Clelland <iclelland@google.com> Date: Fri Jul 07 17:00:00 2017 Fix Disable Feature-Policy HTTP Header for M60 This fixes an issue with commit 360e15ec26cec7d5823421707839e44958678c8b where the wrong capitalization of the generated RuntimeEnabledFeatures method was used. Bug: 737643 Change-Id: I74bb537a33a2b28432470b66c2ff76828f93e563 TBR: rbyers@chromium.org NOTRY: true NOPRESUBMIT: true Reviewed-on: https://chromium-review.googlesource.com/563836 Reviewed-by: Ian Clelland <iclelland@chromium.org> Cr-Commit-Position: refs/branch-heads/3112@{#542} Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897} [modify] https://crrev.com/6ec7521db9dd9304049040de049f3c6819afc8b1/third_party/WebKit/Source/core/loader/DocumentLoader.cpp |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by bugdroid1@chromium.org
, Jun 30 2017