[Bindings] Refactoring of binding code around runtime enabled features

Refactoring of binding code around runtime enabled features

It didn't make sense to use 'runtime_enabled_function_name'
to group runtime enabled features.
Plus, 'runtime_feature_name' was also used in few places. It
was so confusing.

So this CL unify them to use feature names to group contexts
and remove runtime_enabled_function_name() function.
This CL does not change generated code.

BUG=None

Committed: https://crrev.com/9aa18b4eace75d3c11686ce1c030d8511891d914
Cr-Commit-Position: refs/heads/master@{#439088}

[peria, 2016-12-15 09:28:20]

Description was changed from

==========
Refactoring of binding code around runtime enabled features

BUG=None
==========

to

==========
Refactoring of binding code around runtime enabled features

It didn't make sense to use 'runtime_enabled_function_name'
to group runtime enabled features.
Plus, 'runtime_feature_name' was also used in few places. It
was so confusing.

So this CL unify them to use feature names to group contexts
and remove runtime_enabled_function_name() function.
This CL does not change generated code.


BUG=None
==========

[peria, 2016-12-15 10:02:34]

Patchset #1 (id:1) has been deleted

[peria, 2016-12-15 10:04:27]

peria@chromium.org changed reviewers:
+ bashi@chromium.org, haraken@chromium.org, yukishiino@chromium.org

[peria, 2016-12-15 10:04:27]

PTL

[peria, 2016-12-15 10:04:32]

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

[commit-bot: I haz the power, 2016-12-15 10:04:41]

Dry run: 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-12-15 10:12:38]

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

[commit-bot: I haz the power, 2016-12-15 10:12:39]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[haraken, 2016-12-15 11:38:24]

LGTM

[Yuki, 2016-12-15 11:56:39]

https://codereview.chromium.org/2578943002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/scripts/v8_attributes.py (right):

https://codereview.chromium.org/2578943002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/scripts/v8_attributes.py:183:
'runtime_enabled': v8_utilities.runtime_feature_name(attribute),  #
[RuntimeEnabled]
nit: 'runtime_enabled' sounds to me like a boolean value.
I'd prefer runtime_enabled_feature_name or something like that.

[peria, 2016-12-16 05:11:15]

Patchset #2 (id:40001) has been deleted

[peria, 2016-12-16 05:27:04]

https://codereview.chromium.org/2578943002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/scripts/v8_attributes.py (right):

https://codereview.chromium.org/2578943002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/scripts/v8_attributes.py:183:
'runtime_enabled': v8_utilities.runtime_feature_name(attribute),  #
[RuntimeEnabled]
On 2016/12/15 11:56:39, Yuki wrote:
> nit: 'runtime_enabled' sounds to me like a boolean value.
> I'd prefer runtime_enabled_feature_name or something like that.

Done.

[Yuki, 2016-12-16 06:25:25]

LGTM.

[peria, 2016-12-16 09:29:22]

The CQ bit was checked by peria@chromium.org

[peria, 2016-12-16 09:29:22]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2578943002/#ps60001
(title: "Work for a comment")

[commit-bot: I haz the power, 2016-12-16 09:29:38]

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-12-16 11:20:54]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1481880562383050,
"parent_rev": "3c5a406e94a0b9cb34bcb17fb7ea281b6a59bedd", "commit_rev":
"06bad746afcbe608f3ea50eaffb3c8e3ee519775"}

[commit-bot: I haz the power, 2016-12-16 11:21:25]

Description was changed from

==========
Refactoring of binding code around runtime enabled features

It didn't make sense to use 'runtime_enabled_function_name'
to group runtime enabled features.
Plus, 'runtime_feature_name' was also used in few places. It
was so confusing.

So this CL unify them to use feature names to group contexts
and remove runtime_enabled_function_name() function.
This CL does not change generated code.


BUG=None
==========

to

==========
Refactoring of binding code around runtime enabled features

It didn't make sense to use 'runtime_enabled_function_name'
to group runtime enabled features.
Plus, 'runtime_feature_name' was also used in few places. It
was so confusing.

So this CL unify them to use feature names to group contexts
and remove runtime_enabled_function_name() function.
This CL does not change generated code.


BUG=None

Review-Url: https://codereview.chromium.org/2578943002
==========

[commit-bot: I haz the power, 2016-12-16 11:21:26]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2016-12-16 11:23:28]

Description was changed from

==========
Refactoring of binding code around runtime enabled features

It didn't make sense to use 'runtime_enabled_function_name'
to group runtime enabled features.
Plus, 'runtime_feature_name' was also used in few places. It
was so confusing.

So this CL unify them to use feature names to group contexts
and remove runtime_enabled_function_name() function.
This CL does not change generated code.


BUG=None

Review-Url: https://codereview.chromium.org/2578943002
==========

to

==========
Refactoring of binding code around runtime enabled features

It didn't make sense to use 'runtime_enabled_function_name'
to group runtime enabled features.
Plus, 'runtime_feature_name' was also used in few places. It
was so confusing.

So this CL unify them to use feature names to group contexts
and remove runtime_enabled_function_name() function.
This CL does not change generated code.

BUG=None

Committed: https://crrev.com/9aa18b4eace75d3c11686ce1c030d8511891d914
Cr-Commit-Position: refs/heads/master@{#439088}
==========

[commit-bot: I haz the power, 2016-12-16 11:23:29]

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