Support IDL attributes limited to only known values.

Source/bindings/tests/idls/TestObject.idl

Index: Source/bindings/tests/idls/TestObject.idl
diff --git a/Source/bindings/tests/idls/TestObject.idl b/Source/bindings/tests/idls/TestObject.idl
index 984f9baa44c8c24fdfb813578b36955d34642d82..d7893b4a00d904d91bf82661b63d63cac74afef2 100644
--- a/Source/bindings/tests/idls/TestObject.idl
+++ b/Source/bindings/tests/idls/TestObject.idl
@@ -74,6 +74,9 @@ callback TestCallbackFunction = void (TestEnumType formal1, TestObject formal2);
     [Reflect=customContentBooleanAttr] attribute boolean reflectedCustomBooleanAttr;
     [Reflect=customContentURLAttr, TreatNullAs=NullString, URL] attribute DOMString reflectedCustomURLAttr;
     [Reflect] attribute SVGAnimatedString animatedReflectedAttribute;
+    [Reflect, ReflectOnly=unique] attribute DOMString limitedToOnlyOneAttribute;
+    [Reflect, ReflectOnly=Per Paal Espen] attribute DOMString limitedToOnlyAttribute;

[haraken, 2014/01/06 06:23:53]

Shall we use ReflectOnly=Per|Paal|Pspen instead of space splitting?

[sof, 2014/01/06 06:40:51]

I thought I could from reading the IDLExtendedAttributes.txt description for
"Attr=*", but "|" with more than 2 values isn't allowed by the parser.

It looks odd to use space, I agree, should we make the parser more lenient?
(assuming that there isn't a way to express this already.)

[Nils Barth (inactive), 2014/01/06 06:42:51]

??
To agree with haraken, we can't use spaces here.
| should work with arbitrarily many values.
Let me check this.

[Nils Barth (inactive), 2014/01/06 07:12:49]

Thanks for catching this; this is a bug, which I've posted a fix for:
Perl IDL parser: allow extended attributes to have more than 2 values
https://codereview.chromium.org/124943002/

In the initial Python rewrite the grammar explicitly limited extended attribute
value lists to length 2, as that was sufficient for actual use, but I removed
the limit later (someone complained, and it's clearer that way anyway). I didn't
update the Perl then though b/c no-one was using it yet.

[sof, 2014/01/06 10:59:19]

Thanks; rebased this CL on top of it + switched to using "|".

+    [Reflect=Other, ReflectOnly=Value1 Value2] attribute DOMString limitedToOnlyOtherAttribute;
 
     // TypedArray attribute
     attribute Float32Array typedArrayAttr;