Closed Bug 920043 Opened 11 years ago Closed 11 years ago

Update DOM tests from web-platform-tests

Categories

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

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla27
Tracking Status
firefox25 --- wontfix
firefox26 --- fixed
firefox27 --- fixed

People

(Reporter: Ms2ger, Assigned: Ms2ger)

References

Details

(Whiteboard: [qa-])

Attachments

(2 files)

Attached patch Patch v1 (deleted) — Splinter Review
We currently have imported tests from <https://dvcs.w3.org/hg/webapps>, but that repo is no longer maintained in favour of <https://github.com/w3c/web-platform-tests>. At the same time, the directories were reorganized, so the patch is a bit of a mess. This patch also includes changes to make the Range tests be somewhat less oom-happy, so I re-enabled test_Range-surroundContents.html on windows. If it turns out the changes weren't enough, I'll disable it again. https://tbpl.mozilla.org/?tree=Try&rev=db70985105f8 https://tbpl.mozilla.org/?tree=Try&rev=0550c75f44d9 https://tbpl.mozilla.org/?tree=Try&rev=329377a1813d
Attachment #809189 - Flags: review?(ryanvm)
Attachment #809189 - Flags: review?(mounir)
Comment on attachment 809189 [details] [diff] [review] Patch v1 Don't think I'm able to r+ this, but the Try run LGTM :)
Attachment #809189 - Flags: review?(ryanvm) → feedback+
Blocks: 920064
Comment on attachment 809189 [details] [diff] [review] Patch v1 Review of attachment 809189 [details] [diff] [review]: ----------------------------------------------------------------- rs=me
Attachment #809189 - Flags: review?(mounir) → review+
Philor, can you get this dealt with after the tree reopens, if necessary?
Attachment #810100 - Flags: review?(philringnalda)
Attachment #810100 - Flags: review?(philringnalda) → review+
Blocks: 885107
Blocks: 900633
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
Blocks: 894030
Blocks: 858948
With the uplift coming up, I'm going to wontfix this for beta.
Whiteboard: [qa-]
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: