Closed Bug 1298410 Opened 8 years ago Closed 8 years ago

Remove parser support for WebIDL arrays

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla51
Tracking Status
firefox51 --- fixed

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

They are not a thing anymore in the spec.
Comment on attachment 8785339 [details] [diff] [review] Remove support for IDL arrays, since those are not a thing anymore Review of attachment 8785339 [details] [diff] [review]: ----------------------------------------------------------------- ::: dom/bindings/parser/WebIDL.py @@ +4475,5 @@ > 'LegacyCaller', > base=IDLInterfaceMember.Special > ) > > TypeSuffixModifier = enum( Can't you just remove this enum? ::: dom/bindings/parser/tests/test_arraybuffer.py @@ +55,5 @@ > harness.ok(attr.type.isSpiderMonkeyInterface(), "Should test as a js interface") > > (retType, arguments) = method.signatures()[0] > harness.ok(retType.isVoid(), "Should have a void return type") > + harness.check(len(arguments), 3, "Expect 4 arguments") "Expect 3...
Attachment #8785339 - Flags: review?(peterv) → review+
> Can't you just remove this enum? Indeed. Done. > "Expect 3... Also done.
Pushed by bzbarsky@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/b7e5426f358a Remove support for IDL arrays, since those are not a thing anymore. r=peterv
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: