IDL: Drop [PerContextEnabled] extended attribute

IDL: Drop [PerContextEnabled] extended attribute

It is no longer used, and adds complexity to the code generator.

BUG=465624
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=194275

[Jens Widell, 2015-04-23 05:58:05]

jl@opera.com changed reviewers:
+ haraken@chromium.org, yukishiino@chromium.org

[Jens Widell, 2015-04-23 05:58:06]

PTAL

https://codereview.chromium.org/1107523002/diff/1/Source/bindings/templates/m...
File Source/bindings/templates/methods.cpp (right):

https://codereview.chromium.org/1107523002/diff/1/Source/bindings/templates/m...
Source/bindings/templates/methods.cpp:632: {% macro
install_conditionally_enabled_methods() %}
Turns out we can't simply remove this by dropping [PerContextEnabled] since
[Exposed] methods are also conditionally enabled and handled by this code.

[haraken, 2015-04-23 06:01:57]

oh, I didn't know we've already removed customers of [PerContextEnabled]. LGTM.

[Yuki, 2015-04-23 06:04:38]

lgtm.

Thanks for doing this.
I removed the clients of [PerContextEnabled], but have been lazy to do the rest
of work.  Thanks.

[Jens Widell, 2015-04-23 06:07:56]

On 2015/04/23 06:04:38, Yuki wrote:
> Thanks for doing this.
> I removed the clients of [PerContextEnabled], but have been lazy to do the
rest
> of work.  Thanks.

You did all the hard work, I get to do the pleasant code removal/cleanup. I'm
fine with this division of labor. ;-)

[Jens Widell, 2015-04-23 06:08:06]

The CQ bit was checked by jl@opera.com

[commit-bot: I haz the power, 2015-04-23 06:08:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1107523002/1

[Jens Widell, 2015-04-23 06:36:23]

The CQ bit was unchecked by jl@opera.com

[Jens Widell, 2015-04-23 08:14:27]

The CQ bit was checked by jl@opera.com

[Jens Widell, 2015-04-23 08:14:27]

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/1107523002/#ps20001
(title: "rebased")

[commit-bot: I haz the power, 2015-04-23 08:14:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1107523002/20001

[commit-bot: I haz the power, 2015-04-23 09:47:36]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=194275