Beta web platform tests unexpectedly passing with /dom/historical.html | Node member must be nuked: rootNode - expected FAIL

RESOLVED FIXED in Firefox 50

Status

defect
RESOLVED FIXED
3 years ago
10 months ago

People

(Reporter: intermittent-bug-filer, Assigned: cbook)

Tracking

({intermittent-failure})

Version 3
mozilla57
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox50 fixed, firefox51 disabled, firefox52 disabled, firefox57 fixed)

Details

Attachments

(1 attachment)

james, something you can look into ?
Flags: needinfo?(james)
per irc
Flags: needinfo?(james)
Attachment #8792818 - Flags: review?(Ms2ger)
Comment on attachment 8792818 [details] [diff] [review]
bug1303802.patch

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

Thanks
Attachment #8792818 - Flags: review?(Ms2ger) → review+
landed on beta: 

https://hg.mozilla.org/releases/mozilla-beta/rev/a7853dad14c9

thanks ms2ger!
Assignee: nobody → cbook
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Needs to land on trunk/aurora still, unless I'm mistaken.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
How could it? It's an unexamined and probably spurious unexpected pass that only happens on RELEASE_BUILD, can't very well remove the expectation of failure on branches where it still does fail.
This is expected in the sense that it tests that we don't support the rootNode attribute; we currently do support it, but it is preffed off in release builds. There's no need to land anything on trunk.

Bug 1269155 is expected to fix this once and for all.
Blocks: 1269155
Status: REOPENED → RESOLVED
Closed: 3 years ago3 years ago
Resolution: --- → FIXED
What verb tense are you using to call 51 and 52 unaffected, alternate-future-with-different-past? Bug 1269155 hasn't been meaningfully touched since last June, 51 merges to beta on November 7th, and when it does this test will again fail.
Assignee: cbook → nobody
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Blocks: 1307010
Keywords: leave-open
Pushed by philringnalda@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d2193462357a
Disable the part of historical.html which is permaorange on every merge to beta when we disable our rootNode implementation and let the test unexpectedly pass
No longer blocks: 1307010
Can this be re-enabled now?
Flags: needinfo?(philringnalda)
Pushed by philringnalda@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/44edb197a893
reenable rootNode test chunk now that we don't have a rootNode implementation so the rootNode test which failed when we had one now passes when we don't have one
Can't think why not.
Flags: needinfo?(philringnalda)
Keywords: leave-open
https://hg.mozilla.org/mozilla-central/rev/44edb197a893
Status: REOPENED → RESOLVED
Closed: 3 years ago2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Assignee: nobody → cbook
You need to log in before you can comment on or make changes to this bug.