Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(49)

Issue 2484763007: [Feature Policy] Remove cookie, domain, and docwrite from Feature Policy MVP (Closed)

Created:
4 years, 1 month ago by iclelland
Modified:
4 years, 1 month ago
Reviewers:
haraken
CC:
chromium-reviews, sof, eae+blinkwatch, blink-reviews-dom_chromium.org, dglazkov+blink, blink-reviews-bindings_chromium.org, blink-reviews, rwlbuis
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[Feature Policy] Remove cookie, domain, and docwrite from Feature Policy MVP These three features are not shipping as part of the MVP for Feature Policy, and so should be removed for now. Eventually they can be reinstanted (behind a flag) for experimentation and eventual shipping as part of FP V1. BUG=663817 Committed: https://crrev.com/d84b70cf3a1adbd5d861ee1f8cfc81d94437215d Cr-Commit-Position: refs/heads/master@{#431183}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -24 lines) Patch
M third_party/WebKit/Source/bindings/core/v8/ConditionalFeatures.cpp View 1 chunk +0 lines, -20 lines 0 comments Download
M third_party/WebKit/Source/core/dom/Document.idl View 2 chunks +4 lines, -4 lines 0 comments Download

Messages

Total messages: 12 (7 generated)
iclelland
+r haraken, can you PTAL? This effectively removes the only IDL features from feature policy. ...
4 years, 1 month ago (2016-11-09 19:54:48 UTC) #4
haraken
On 2016/11/09 19:54:48, iclelland wrote: > +r haraken, can you PTAL? > > This effectively ...
4 years, 1 month ago (2016-11-10 02:08:37 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2484763007/1
4 years, 1 month ago (2016-11-10 04:40:24 UTC) #9
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 1 month ago (2016-11-10 04:45:19 UTC) #10
commit-bot: I haz the power
4 years, 1 month ago (2016-11-10 04:48:14 UTC) #12
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/d84b70cf3a1adbd5d861ee1f8cfc81d94437215d
Cr-Commit-Position: refs/heads/master@{#431183}

Powered by Google App Engine
This is Rietveld 408576698