Add IDL parsing of FeaturePolicy extended attribute

Add IDL parsing of FeaturePolicy extended attribute

This is an interim step to handle FeaturePolicy-enabled features, before the
three conditional feature flags (RuntimeEnabled, OriginTrialEnabled and
FeaturePolicy) are eventually combined into a common 'Feature' attribute.

BUG=638240
Committed: https://crrev.com/60deeff69759a9fe496984280386a8d1213fd797
Cr-Commit-Position: refs/heads/master@{#412524}

[iclelland, 2016-08-16 15:03:01]

The CQ bit was checked by iclelland@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-16 15:03:15]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[iclelland, 2016-08-16 15:08:34]

Description was changed from

==========
Add IDL parsing of FeaturePolicy extended attribute

This is an interim step to handle FeaturePolicy-enabled features, before the
three conditional feature flags (RuntimeEnabled, OriginTrialEnabled and
FeaturePolicy) are eventually combined into a common 'Feature' attribute.

BUG=638240
==========

to

==========
Add IDL parsing of FeaturePolicy extended attribute

This is an interim step to handle FeaturePolicy-enabled features, before the
three conditional feature flags (RuntimeEnabled, OriginTrialEnabled and
FeaturePolicy) are eventually combined into a common 'Feature' attribute.

BUG=638240
==========

[iclelland, 2016-08-16 15:08:34]

iclelland@chromium.org changed reviewers:
+ yukishiino@chromium.org

[iclelland, 2016-08-16 15:08:52]

+r Yuki -- can you PTAL? Thanks!

[commit-bot: I haz the power, 2016-08-16 17:23:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-16 17:23:39]

Dry run: This issue passed the CQ dry run.

[haraken, 2016-08-17 01:18:43]

haraken@chromium.org changed reviewers:
+ haraken@chromium.org

[haraken, 2016-08-17 01:18:44]

LGTM

> before the three conditional feature flags (RuntimeEnabled, OriginTrialEnabled
and
> FeaturePolicy) are eventually combined into a common 'Feature' attribute.

Yeah, I want to make this happen. Do you have any timeline for the unification?

https://codereview.chromium.org/2247923004/diff/1/third_party/WebKit/Source/b...
File third_party/WebKit/Source/build/scripts/make_runtime_features.py (right):

https://codereview.chromium.org/2247923004/diff/1/third_party/WebKit/Source/b...
third_party/WebKit/Source/build/scripts/make_runtime_features.py:46:
'origin_trial_feature_name': None,

Would you sort this dictionary in the alphabetical order?

[Yuki, 2016-08-17 03:16:46]

lgtm

[iclelland, 2016-08-17 13:25:55]

> Yeah, I want to make this happen. Do you have any timeline for the
unification?

I'm still hoping for end-of-Q3 for this

https://codereview.chromium.org/2247923004/diff/1/third_party/WebKit/Source/b...
File third_party/WebKit/Source/build/scripts/make_runtime_features.py (right):

https://codereview.chromium.org/2247923004/diff/1/third_party/WebKit/Source/b...
third_party/WebKit/Source/build/scripts/make_runtime_features.py:46:
'origin_trial_feature_name': None,
On 2016/08/17 01:18:43, haraken wrote:
> 
> Would you sort this dictionary in the alphabetical order?

Done.

[iclelland, 2016-08-17 13:26:21]

The CQ bit was checked by iclelland@chromium.org

[iclelland, 2016-08-17 13:26:22]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, yukishiino@chromium.org
Link to the patchset: https://codereview.chromium.org/2247923004/#ps20001
(title: "Alphabetize defaults list in make_runtime_features.py")

[commit-bot: I haz the power, 2016-08-17 13:26:39]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-17 14:44:11]

Description was changed from

==========
Add IDL parsing of FeaturePolicy extended attribute

This is an interim step to handle FeaturePolicy-enabled features, before the
three conditional feature flags (RuntimeEnabled, OriginTrialEnabled and
FeaturePolicy) are eventually combined into a common 'Feature' attribute.

BUG=638240
==========

to

==========
Add IDL parsing of FeaturePolicy extended attribute

This is an interim step to handle FeaturePolicy-enabled features, before the
three conditional feature flags (RuntimeEnabled, OriginTrialEnabled and
FeaturePolicy) are eventually combined into a common 'Feature' attribute.

BUG=638240
==========

[commit-bot: I haz the power, 2016-08-17 14:44:12]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-17 14:45:27]

Description was changed from

==========
Add IDL parsing of FeaturePolicy extended attribute

This is an interim step to handle FeaturePolicy-enabled features, before the
three conditional feature flags (RuntimeEnabled, OriginTrialEnabled and
FeaturePolicy) are eventually combined into a common 'Feature' attribute.

BUG=638240
==========

to

==========
Add IDL parsing of FeaturePolicy extended attribute

This is an interim step to handle FeaturePolicy-enabled features, before the
three conditional feature flags (RuntimeEnabled, OriginTrialEnabled and
FeaturePolicy) are eventually combined into a common 'Feature' attribute.

BUG=638240

Committed: https://crrev.com/60deeff69759a9fe496984280386a8d1213fd797
Cr-Commit-Position: refs/heads/master@{#412524}
==========

[commit-bot: I haz the power, 2016-08-17 14:45:28]

Patchset 2 (id:??) landed as
https://crrev.com/60deeff69759a9fe496984280386a8d1213fd797
Cr-Commit-Position: refs/heads/master@{#412524}