Parser complains about NamedConstructor and NoInterfaceObject being incompatible, spec allows it

RESOLVED FIXED in mozilla23

Status

()

Core
DOM
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: reuben, Assigned: reuben)

Tracking

Trunk
mozilla23
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
[NoInterfaceObject,
 NamedConstructor=mozFoo]
interface Foo { }

WebIDL.WebIDLError: error: NamedConstructor and NoInterfaceObject are incompatible
(Assignee)

Comment 1

5 years ago
Created attachment 740572 [details] [diff] [review]
Fix
Assignee: nobody → reuben.bmo
Attachment #740572 - Flags: review?(bzbarsky)
(Assignee)

Comment 2

5 years ago
Created attachment 740573 [details] [diff] [review]
Patch, minus whitespace hell

Sorry about that.
Attachment #740572 - Attachment is obsolete: true
Attachment #740572 - Flags: review?(bzbarsky)
Attachment #740573 - Flags: review?(bzbarsky)
Comment on attachment 740573 [details] [diff] [review]
Patch, minus whitespace hell

For that last test, you don't need the try/catch or threw variable.  Just run the code, and if it throws the test will fail.

But you _do_ need to do |parser = parser.reset()|.

r=me with that and with the missing newline at EOF added.
Attachment #740573 - Flags: review?(bzbarsky) → review+
Er, you need the reset() _before_ your test, to reset it from the previous test....
Nah, I'm just sorry I didn't make that clearer.  :(

Comment 8

5 years ago
https://hg.mozilla.org/mozilla-central/rev/138187d07a1c
https://hg.mozilla.org/mozilla-central/rev/f1f6be722f2a
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.