Closed Bug 1215812 Opened 6 years ago Closed 5 years ago

Intermittent TEST-UNEXPECTED-FAIL browsing-context-names/002.html | Link with target=_blank, no rel - assert_unreached: Failed to get callback from opened window Reached unreachable code

Categories

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

Product:
Core
Component:
DOM: Core & HTML
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla48
Tracking Flags:
Tracking Status
firefox44 --- wontfix
firefox45 --- wontfix
firefox46 --- fixed
firefox47 --- fixed
firefox48 --- fixed
firefox-esr45 --- fixed

People

(Reporter: aryx, Assigned: jgraham)

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

MozReview Request: Bug 1215812 - Increase the internal timeout for a test that depends on a window opening, r=Ms2ger
58 bytes, text/x-review-board-request
Ms2ger
: review+
Details 
02:24:47     INFO - TEST-START | /html/browsers/windows/browsing-context-names/002.html
02:24:48     INFO - TEST-UNEXPECTED-FAIL | /html/browsers/windows/browsing-context-names/002.html | Link with target=_blank, no rel - assert_unreached: Failed to get callback from opened window Reached unreachable code
02:24:48     INFO - @http://web-platform.test:8000/html/browsers/windows/browsing-context-names/002.html:17:16
02:24:48     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1380:20
02:24:48     INFO - Test.prototype.step_func/<@http://web-platform.test:8000/resources/testharness.js:1404:1
02:24:48     INFO - setTimeout handler*@http://web-platform.test:8000/html/browsers/windows/browsing-context-names/002.html:16:1
02:24:48     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1380:20
02:24:48     INFO - @http://web-platform.test:8000/html/browsers/windows/browsing-context-names/002.html:10:1
02:24:48     INFO - TEST-OK | /html/browsers/windows/browsing-context-names/002.html | took 1338ms
This has been failing intermittently pretty consistently since October, but I'm really not sure who's the right person to look at this. Can you please help, Andrew?
Flags: needinfo?(overholt)
khuey and I just looked at this. Is it possible the 1 s timeout [1] is just too short?

[1]
https://dxr.mozilla.org/mozilla-central/source/testing/web-platform/tests/html/browsers/windows/browsing-context-names/002.html#18
Flags: needinfo?(overholt) → needinfo?(ryanvm)
I have no idea who created this test. James?
Flags: needinfo?(ryanvm) → needinfo?(james)
Yeah, I imagine it's just an over-optimistic timer. I will Try a fix.
Flags: needinfo?(james)
Comment on attachment 8729525 [details]
MozReview Request: Bug 1215812 - Increase the internal timeout for a test that depends on a window opening, r=Ms2ger

https://reviewboard.mozilla.org/r/39481/#review36319
Attachment #8729525 - Flags: review?(Ms2ger) → review+
https://hg.mozilla.org/mozilla-central/rev/9814ff026953
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox48: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.