[Bug 25994] New: invalid extended attribute list syntax

https://www.w3.org/Bugs/Public/show_bug.cgi?id=25994

            Bug ID: 25994
           Summary: invalid extended attribute list syntax
           Product: WebAppsWG
           Version: unspecified
          Hardware: All
                OS: All
            Status: NEW
          Severity: critical
          Priority: P2
         Component: File API
          Assignee: arun@mozilla.com
          Reporter: glenn@skynav.com
        QA Contact: public-webapps-bugzilla@w3.org
                CC: public-webapps@w3.org

The extended attribute list syntax used in the File API does not adhere to
current WebIDL specification syntax. In particular, production 64 of the syntax
[1] must consist of a comma separated list of ExtendedAttribute items.

[1] http://heycam.github.io/webidl/#proddef-ExtendedAttributeList

In order to fix this, please make the following changes:

(1) on definition of Blob interface, change

[Exposed=Window,Worker][Constructor,
Constructor(sequence<(ArrayBuffer or ArrayBufferView or Blob or DOMString)>
blobParts, optional BlobPropertyBag options)]

to 

[Constructor,
Constructor(sequence<(ArrayBuffer or ArrayBufferView or Blob or DOMString)>
blobParts, optional BlobPropertyBag options),
Exposed=Window,Worker]

(2) on definition of File interface, change

[Exposed=Window,Worker][Constructor(sequence<(Blob or DOMString or
ArrayBufferView or ArrayBuffer)> fileBits, [EnsureUTF16] DOMString fileName,
optional FilePropertyBag options)]

to

[Constructor(sequence<(Blob or DOMString or ArrayBufferView or ArrayBuffer)>
fileBits, [EnsureUTF16] DOMString fileName, optional FilePropertyBag options),
Exposed=Window,Worker]

(3) on definition of FileReader interface, change

[Exposed=Window,Worker][Constructor]

to

[Constructor, Exposed=Window,Worker]

(4) on definition of FileReaderSync interface, change

[Exposed=Worker][Constructor]

to

[Constructor, Exposed=Worker]

Note that I have also reordered the Exposed extended attribute above to improve
readability (given that it can contain a ',' as well).

-- 
You are receiving this mail because:
You are on the CC list for the bug.

Received on Thursday, 5 June 2014 17:08:47 UTC