Fix DOM test_interfaces.html

RESOLVED FIXED in mozilla27

Status

()

Core
DOM
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Tracking

unspecified
mozilla27
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
Created attachment 809214 [details] [diff] [review]
Patch v1

The parsing fails because the version of the IDL parser we have doesn't support default arguments. Since these aren't actually used, I'm just going to comment them out until we update the IDL parser.

I also wrapped the parsing in a setup call, so we won't have this silently happen again (upstreamed as <https://github.com/w3c/web-platform-tests/pull/351>).
Attachment #809214 - Flags: review?(james)
Attachment #809214 - Flags: review?(james) → review+
(Assignee)

Comment 1

4 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/0a2182931c50
Flags: in-testsuite-
Comment on attachment 809214 [details] [diff] [review]
Patch v1

>-  void collapse(optional boolean toStart = false);
>+  void collapse(optional boolean toStart /* = false */);

Isn't it a read-only copy of the upstream? :)
https://hg.mozilla.org/mozilla-central/rev/0a2182931c50
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in before you can comment on or make changes to this bug.