IDL compiler: [CustomElementCallbacks] for getter + cleanup values

IDL compiler: [CustomElementCallbacks] for getter + cleanup values

[CustomElementCallbacks] is only ever called as [CustomElementCallbacks=Enable],
so the value is redundant.
This changes it to not take an argument.
Also the Perl CG has cruft from when 'Disable' was called 'None';
this decrufts it.

This also adds a test case to TestObjectPython.idl, though since this
extended attribute doesn't affect getters, no Python CG changes are needed;
will do when we do setters.

Doc:
https://sites.google.com/a/chromium.org/dev/blink/webidl/blink-idl-extended-attributes#TOC-CustomElementCallbacks-m-a-

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

[Nils Barth (inactive), 2013-10-08 11:01:54]

Mostly Perl CG/IDL extended attribute fixes.
(No Python CG changes, just test.)

[haraken, 2013-10-08 11:06:51]

https://codereview.chromium.org/26466003/diff/3001/Source/bindings/tests/resu...
File Source/bindings/tests/results/V8TestObjectPython.cpp (right):

https://codereview.chromium.org/26466003/diff/3001/Source/bindings/tests/resu...
Source/bindings/tests/results/V8TestObjectPython.cpp:845: static void
customElementsCallbacksReadonlyLongAttributeAttributeGetterCallback(v8::Local<v8::String>
name, const v8::PropertyCallbackInfo<v8::Value>& info)

Is this generated code correct? I expected:

  CustomElementCallbackDispatcher::CallbackDeliveryScope deliveryScope;

is generated.

[Nils Barth (inactive), 2013-10-09 00:43:36]

https://codereview.chromium.org/26466003/diff/3001/Source/bindings/tests/resu...
File Source/bindings/tests/results/V8TestObjectPython.cpp (right):

https://codereview.chromium.org/26466003/diff/3001/Source/bindings/tests/resu...
Source/bindings/tests/results/V8TestObjectPython.cpp:845: static void
customElementsCallbacksReadonlyLongAttributeAttributeGetterCallback(v8::Local<v8::String>
name, const v8::PropertyCallbackInfo<v8::Value>& info)
On 2013/10/08 11:06:51, haraken wrote:
> 
> Is this generated code correct? I expected:
> 
>   CustomElementCallbackDispatcher::CallbackDeliveryScope deliveryScope;
> 
> is generated.

This is what the Perl generates.
It only adds that line, i.e., calls:
GenerateCustomElementInvocationScopeIfNeeded
...for setters, setter callbacks, and functions, but not for getters.

[haraken, 2013-10-09 00:46:20]

Thanks for clarification. LGTM.

[Nils Barth (inactive), 2013-10-09 00:50:47]

On 2013/10/09 00:46:20, haraken wrote:
> Thanks for clarification. LGTM.

n/p, committing!

[commit-bot: I haz the power, 2013-10-09 00:50:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nbarth@chromium.org/26466003/3001

[commit-bot: I haz the power, 2013-10-09 11:09:18]

Change committed as 159218