Closed
Bug 920043
Opened 10 years ago
Closed 10 years ago
Update DOM tests from web-platform-tests
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla27
People
(Reporter: Ms2ger, Assigned: Ms2ger)
References
Details
(Whiteboard: [qa-])
Attachments
(2 files)
174.70 KB,
patch
|
mounir
:
review+
RyanVM
:
feedback+
|
Details | Diff | Splinter Review |
2.71 KB,
patch
|
philor
:
review+
|
Details | Diff | 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 1•10 years ago
|
||
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+
Comment 2•10 years ago
|
||
Comment on attachment 809189 [details] [diff] [review] Patch v1 Review of attachment 809189 [details] [diff] [review]: ----------------------------------------------------------------- rs=me
Attachment #809189 -
Flags: review?(mounir) → review+
Assignee | ||
Comment 3•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/75c782b0ec27
Flags: in-testsuite-
Assignee | ||
Comment 4•10 years ago
|
||
Philor, can you get this dealt with after the tree reopens, if necessary?
Attachment #810100 -
Flags: review?(philringnalda)
Updated•10 years ago
|
Attachment #810100 -
Flags: review?(philringnalda) → review+
Updated•10 years ago
|
Keywords: checkin-needed
Comment 6•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/3b5b0877e4e4
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/75c782b0ec27
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
Comment 8•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/3b5b0877e4e4
Comment 9•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/53cb95cfe6dc
status-firefox26:
--- → fixed
status-firefox27:
--- → fixed
Assignee | ||
Comment 10•10 years ago
|
||
With the uplift coming up, I'm going to wontfix this for beta.
status-firefox25:
--- → wontfix
Updated•10 years ago
|
Whiteboard: [qa-]
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•