Update layout/mathml/tests/various.xml to use the new maction syntax

RESOLVED FIXED in mozilla16

Status

()

Core
MathML
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: fredw, Assigned: fredw)

Tracking

(Depends on: 1 bug)

Trunk
mozilla16
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(3 attachments, 2 obsolete attachments)

(Assignee)

Description

5 years ago
The test page

layout/mathml/tests/various.xml

contains various maction elements that should be updated to use our new syntax: we removed restyle actiontype and changed statusline actiontype to use the standard MathML3 syntax.
(Assignee)

Updated

5 years ago
Whiteboard: [mentor=fredw] → [mentor=fredw][lang=xml]
(Assignee)

Comment 1

5 years ago
I'm going to clean up a bit layout/mathml/tests/ at the same time. These tests are not really used and most of them have equivalent constructions in the MathML demo pages. I'll just leave some tests that we'll probably move later to these MathML demo pages.
Assignee: nobody → fred.wang
Status: NEW → ASSIGNED
Keywords: helpwanted
Whiteboard: [mentor=fredw][lang=xml]
(Assignee)

Comment 2

5 years ago
Created attachment 629312 [details] [diff] [review]
Part 1
Attachment #629312 - Flags: review?(karlt)
(Assignee)

Comment 3

5 years ago
Created attachment 629313 [details] [diff] [review]
Remove some tests from layout/mathml/tests/various.xml + merge it with maxwell.xml
Attachment #629313 - Flags: review?(karlt)
(Assignee)

Comment 4

5 years ago
Created attachment 629314 [details] [diff] [review]
Remove some tests from layout/mathml/tests/
Attachment #629314 - Flags: review?(karlt)
Attachment #629312 - Flags: review?(karlt) → review+
Attachment #629313 - Flags: review?(karlt) → review+
Attachment #629314 - Flags: superreview+
Attachment #629314 - Flags: superreview+
Attachment #629314 - Flags: review?(karlt)
Attachment #629314 - Flags: review+
(Assignee)

Updated

5 years ago
Attachment #629312 - Attachment description: Update layout/mathml/tests/various.xml to use the new maction syntax → Part 1
(Assignee)

Comment 5

5 years ago
Created attachment 630113 [details] [diff] [review]
Part 2
Attachment #629313 - Attachment is obsolete: true
Attachment #629314 - Attachment is obsolete: true
(Assignee)

Comment 6

5 years ago
Created attachment 630114 [details] [diff] [review]
Part 3
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/f934a22d82c9
https://hg.mozilla.org/integration/mozilla-inbound/rev/e2091a32ca5e
https://hg.mozilla.org/integration/mozilla-inbound/rev/b644ec7f0b17
Flags: in-testsuite+
Keywords: checkin-needed
Target Milestone: --- → mozilla16
https://hg.mozilla.org/mozilla-central/rev/f934a22d82c9
https://hg.mozilla.org/mozilla-central/rev/e2091a32ca5e
https://hg.mozilla.org/mozilla-central/rev/b644ec7f0b17
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.