Closed Bug 838088 Opened 11 years ago Closed 11 years ago

Update html tests

Categories

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

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla21

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Patch v1 (obsolete) — Splinter Review
Now the HTML WG has moved to github and moved around most of the tests, we should update. As a bonus, we get a few extra tests.
Attachment #710082 - Flags: review?(mounir)
Comment on attachment 710082 [details] [diff] [review]
Patch v1

I'll get back to you next week.
Attachment #710082 - Flags: review?(mounir)
(In reply to :Ms2ger from comment #1)
> Comment on attachment 710082 [details] [diff] [review]
> Patch v1
> 
> I'll get back to you next week.

Looking forward to it ;)
Attached patch Patch v2Splinter Review
Attachment #712415 - Flags: review?(mounir)
Attachment #710082 - Attachment is obsolete: true
This patch also fixes the incorrect test for <time>'s microdata API.
Comment on attachment 712415 [details] [diff] [review]
Patch v2

Review of attachment 712415 [details] [diff] [review]:
-----------------------------------------------------------------

rs=me
Attachment #712415 - Flags: review?(mounir) → review+
https://hg.mozilla.org/mozilla-central/rev/43625280ba71
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Looks like at some point, the window.onerror tests started passing, so I didn't notice that I accidentally enabled them when moving those tests around.

https://hg.mozilla.org/mozilla-central/rev/c232bec6974d
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: