idl: Fix [EnforceRange] annotations

This CL fixes usages of [EnforceRange] annotations in IDL files.
No changes on behaviors.

CSSUnitValue:
 [EnforceRange] can't annotate non-integer numbers, and the spec
 doesn't have the annotation.

OffscreenCanvas:
 The spec doesn't have [EnforceRange], but it seems the spec is wrong.
 (requesting to update the spec; https://github.com/whatwg/html/issues/3540)

IDBFactory and IDBIndex:
 Follows spec.

Test IDLs:
 This CL works for tests in Chromium repository, and following tests
 need to be upstream for WPT.
 - wpt/interfaces/IndexedDB.idl
 - wpt/interfaces/WebCryptoAPI.idl


Bug: 819112
Change-Id: If7082cba0251a83e269fea428340b3b61d4c17cf
Reviewed-on: https://chromium-review.googlesource.com/952589Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
Reviewed-by: Kent Tamura <tkent@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Hitoshi Yoshida <peria@chromium.org>
Cr-Commit-Position: refs/heads/master@{#542375}

third_party/WebKit/LayoutTests/external/wpt/interfaces/IndexedDB.idl

@@ -42,7 +42,7 @@ partial interface WindowOrWorkerGlobalScope {
 [Exposed=(Window,Worker)]
 interface IDBFactory {
   IDBOpenDBRequest open(DOMString name,
-                        [EnforceRange] optional unsigned long long version);
+                        optional [EnforceRange] unsigned long long version);
   IDBOpenDBRequest deleteDatabase(DOMString name);
 
   short cmp(any first, any second);

third_party/WebKit/LayoutTests/external/wpt/interfaces/WebCryptoAPI.idl

@@ -131,7 +131,7 @@ dictionary CryptoKeyPair {
 
 dictionary RsaKeyGenParams : Algorithm {
   // The length, in bits, of the RSA modulus
-  [EnforceRange] required unsigned long modulusLength;
+  required [EnforceRange] unsigned long modulusLength;
   // The RSA public exponent
   required BigInteger publicExponent;
 };

third_party/WebKit/Source/bindings/tests/idls/core/TestObject.idl

@@ -149,7 +149,7 @@ interface TestObject {
     [Custom=Getter] readonly attribute object customGetterReadonlyObjectAttribute;
     [Custom=Setter] attribute long customSetterLongAttribute;
     [DeprecateAs=LongAttribute] attribute long deprecatedLongAttribute;
-    [EnforceRange] attribute long enforceRangeLongAttribute;
+    attribute [EnforceRange] long enforceRangeLongAttribute;
     [ImplementedAs=implementedAsName] attribute long implementedAsLongAttribute;
     [Custom, ImplementedAs=implementedAsNameWithCustom] attribute long customImplementedAsLongAttribute;
     [Custom=Getter, ImplementedAs=implementedAsNameWithCustomGetter] attribute long customGetterImplementedAsLongAttribute;

third_party/WebKit/Source/core/css/cssom/CSSUnitValue.idl

@@ -11,6 +11,6 @@
   RuntimeEnabled=CSSTypedOM,
   Exposed=(Window,LayoutWorklet,PaintWorklet)
 ] interface CSSUnitValue : CSSNumericValue {
-  [EnforceRange] attribute double value;
+  attribute double value;
   readonly attribute DOMString unit;
 };

third_party/WebKit/Source/core/offscreencanvas/OffscreenCanvas.idl

@@ -10,8 +10,8 @@
     RuntimeEnabled=OffscreenCanvas,
     MeasureAs=OffscreenCanvas
 ] interface OffscreenCanvas : EventTarget {
-    [EnforceRange] attribute unsigned long width;
-    [EnforceRange] attribute unsigned long height;
+    attribute [EnforceRange] unsigned long width;
+    attribute [EnforceRange] unsigned long height;
 
     [CallWith=ScriptState, RaisesException] ImageBitmap transferToImageBitmap();
     [MeasureAs=OffscreenCanvasConvertToBlob, RaisesException, CallWith=ScriptState] Promise<Blob> convertToBlob(optional ImageEncodeOptions options);

third_party/WebKit/Source/core/testing/TypeConversions.idl

@@ -25,24 +25,24 @@
 
 interface TypeConversions {
     attribute long testLong;
-    [EnforceRange, ImplementedAs=testLong] attribute long testEnforceRangeLong;
+    [ImplementedAs=testLong] attribute [EnforceRange] long testEnforceRangeLong;
     attribute unsigned long testUnsignedLong;
-    [EnforceRange, ImplementedAs=testUnsignedLong] attribute unsigned long testEnforceRangeUnsignedLong;
+    [ImplementedAs=testUnsignedLong] attribute [EnforceRange] unsigned long testEnforceRangeUnsignedLong;
 
     attribute long long testLongLong;
-    [EnforceRange, ImplementedAs=testLongLong] attribute long long testEnforceRangeLongLong;
+    [ImplementedAs=testLongLong] attribute [EnforceRange] long long testEnforceRangeLongLong;
     attribute unsigned long long testUnsignedLongLong;
-    [EnforceRange, ImplementedAs=testUnsignedLongLong] attribute unsigned long long testEnforceRangeUnsignedLongLong;
+    [ImplementedAs=testUnsignedLongLong] attribute [EnforceRange] unsigned long long testEnforceRangeUnsignedLongLong;
 
     attribute byte testByte;
-    [EnforceRange, ImplementedAs=testByte] attribute byte testEnforceRangeByte;
+    [ImplementedAs=testByte] attribute [EnforceRange] byte testEnforceRangeByte;
     attribute octet testOctet;
-    [EnforceRange, ImplementedAs=testOctet] attribute octet testEnforceRangeOctet;
+    [ImplementedAs=testOctet] attribute [EnforceRange] octet testEnforceRangeOctet;
 
     attribute short testShort;
-    [EnforceRange, ImplementedAs=testShort] attribute short testEnforceRangeShort;
+    [ImplementedAs=testShort] attribute [EnforceRange] short testEnforceRangeShort;
     attribute unsigned short testUnsignedShort;
-    [EnforceRange, ImplementedAs=testUnsignedShort] attribute unsigned short testEnforceRangeUnsignedShort;
+    [ImplementedAs=testUnsignedShort] attribute [EnforceRange] unsigned short testEnforceRangeUnsignedShort;
 
     attribute ByteString testByteString;
     void setTestByteString(ByteString byteString);

third_party/WebKit/Source/modules/indexeddb/IDBFactory.idl

@@ -29,7 +29,7 @@
     Exposed=(Window,Worker)
 ] interface IDBFactory {
     [NewObject, CallWith=ScriptState, RaisesException] IDBOpenDBRequest open(DOMString name,
-                                                                             [EnforceRange] optional unsigned long long version);
+                                                                             optional [EnforceRange] unsigned long long version);
     [NewObject, CallWith=ScriptState, RaisesException] IDBOpenDBRequest deleteDatabase(DOMString name);
 
     [CallWith=ScriptState, RaisesException] short cmp(any first, any second);

third_party/WebKit/Source/modules/indexeddb/IDBIndex.idl

@@ -37,9 +37,9 @@
     [NewObject, CallWith=ScriptState, RaisesException] IDBRequest get(any key);
     [NewObject, CallWith=ScriptState, RaisesException] IDBRequest getKey(any key);
     [NewObject, CallWith=ScriptState, RaisesException] IDBRequest getAll([Default=Undefined] optional any query,
-                                                                         [EnforceRange] optional unsigned long count);
+                                                                         optional [EnforceRange] unsigned long count);
     [NewObject, CallWith=ScriptState, RaisesException] IDBRequest getAllKeys([Default=Undefined] optional any query,
-                                                                             [EnforceRange] optional unsigned long count);
+                                                                             optional [EnforceRange] unsigned long count);
     [NewObject, CallWith=ScriptState, RaisesException] IDBRequest count([Default=Undefined] optional any key);
 
     [NewObject, CallWith=ScriptState, RaisesException] IDBRequest openCursor([Default=Undefined] optional any range,