Support missing and invalid defaults for reflected attributes.

Support missing and invalid defaults for reflected attributes.

Some enumerated attributes have a designated state if the underlying
attribute value is either missing or in an invalid state. That state
should be observable for reflected IDL attributes on getting, hence
add support for controlling what values to use for these attribute
states:

  [ReflectMissing=missingValue]
  [ReflectInvalid=invalidValue]

The attributes only have an interpretation for limited DOMString
attributes that use [ReflectOnly=<list>]. The missing/invalid value
defaults must appear in the ReflectOnly list.

R=
BUG=331694
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=164579

[sof, 2014-01-07 10:39:02]

Please take a look when you next have a spare moment.

2nd (and final) reflection attribute-related CL. Indiscriminately adding new
attributes isn't a worthwhile goal, but the alternative of encoding this in the
token values for [ReflectOnly=<list>] seemed like more trouble & on the obscure
side. (e.g., prefix a value with "*" or some such to signal a special
interpretation.)

[haraken, 2014-01-07 10:48:18]

LGTM

https://codereview.chromium.org/98033003/diff/1/Source/bindings/scripts/code_...
File Source/bindings/scripts/code_generator_v8.pm (right):

https://codereview.chromium.org/98033003/diff/1/Source/bindings/scripts/code_...
Source/bindings/scripts/code_generator_v8.pm:6269: if (!($missingValueDefault ~~
@knownValues)) {

Nit: I'd prefer not using ~~, since it won't be popular for non-Perler.

[sof, 2014-01-07 11:40:52]

https://codereview.chromium.org/98033003/diff/1/Source/bindings/scripts/code_...
File Source/bindings/scripts/code_generator_v8.pm (right):

https://codereview.chromium.org/98033003/diff/1/Source/bindings/scripts/code_...
Source/bindings/scripts/code_generator_v8.pm:6269: if (!($missingValueDefault ~~
@knownValues)) {
On 2014/01/07 10:48:19, haraken wrote:
> 
> Nit: I'd prefer not using ~~, since it won't be popular for non-Perler.

Thanks, switched to the grep {..} idiom used elsewhere in this code.

[commit-bot: I haz the power, 2014-01-07 11:41:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sigbjornf@opera.com/98033003/60001

[commit-bot: I haz the power, 2014-01-07 13:17:08]

Change committed as 164579

[Nils Barth (inactive), 2014-02-05 06:30:56]

Python side in:
IDL compiler: [ReflectMissing], [ReflectInvalid]
https://codereview.chromium.org/152043003/