Open Bug 1081925 Opened 10 years ago Updated 10 months ago

Intermittent browser_popup_blocker.js | Test timed out - expected PASS

Categories

(Toolkit :: Popup Blocker, defect, P3)

Product:
Toolkit
Component:
Popup Blocker
Platform:
x86
Linux
Type:
defect

Tracking

()

People

(Reporter: cbook, Unassigned)

References

(
URL
)

Details

(Keywords: intermittent-failure, Whiteboard: [test disabled on linux][leave open]) 

Ubuntu ASAN VM 12.04 x64 mozilla-inbound opt test mochitest-e10s-browser-chrome-1

https://treeherder.mozilla.org/ui/logviewer.html#?job_id=2956118&repo=mozilla-inbound

04:29:40 INFO - 426 INFO TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/general/browser_popup_blocker.js | Test timed out - expected PASS
This and bug 1118277 are forming a pretty potent combination of fail at the moment.
Flags: needinfo?(ttaubert)
See Also: → 1118277
Combined, the two bugs would be #3 overall on OrangeFactor.
did some retriggers:
https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&fromchange=0055add82982&tochange=c6a3db6682e0&filter-searchStr=Ubuntu%20VM%2012.04%20x64%20mozilla-inbound%20debug%20test%20mochitest-browser-chrome-1&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-resultStatus=retry&filter-resultStatus=usercancel&filter-resultStatus=running&filter-resultStatus=pending

and the winner is:
https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=9fda841228ac

and this is just a test change:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9fda841228ac

as this enabled a test for e10s mode, by fixing the test and removing the manifest condition, it shouldn't affect non e10s, but in this case it is causing a future test to have troubles.

maybe Neil can shed some light on the changes he made and why we are having issues in a future test.  it might be best to back it out while we debug on try unless there is something obvious.  We have 3-4 failures/20 jobs on average.  (as a note, I am referring to this error and the one in bug 1118277)
Flags: needinfo?(ttaubert) → needinfo?(enndeakin)
(In reply to Joel Maher (:jmaher) from comment #126)
> did some retriggers:
> https://treeherder.mozilla.org/#/jobs?repo=mozilla-
> inbound&fromchange=0055add82982&tochange=c6a3db6682e0&filter-
> searchStr=Ubuntu%20VM%2012.04%20x64%20mozilla-
> inbound%20debug%20test%20mochitest-browser-chrome-1&filter-
> resultStatus=testfailed&filter-resultStatus=busted&filter-
> resultStatus=exception&filter-resultStatus=retry&filter-
> resultStatus=usercancel&filter-resultStatus=running&filter-
> resultStatus=pending
> 
> and the winner is:
> https://treeherder.mozilla.org/#/jobs?repo=mozilla-
> inbound&revision=9fda841228ac
> 
> and this is just a test change:
> https://hg.mozilla.org/integration/mozilla-inbound/rev/9fda841228ac
> 
> as this enabled a test for e10s mode, by fixing the test and removing the
> manifest condition, it shouldn't affect non e10s, but in this case it is
> causing a future test to have troubles.
> 
> maybe Neil can shed some light on the changes he made and why we are having
> issues in a future test.  it might be best to back it out while we debug on
> try unless there is something obvious.  We have 3-4 failures/20 jobs on
> average.  (as a note, I am referring to this error and the one in bug
> 1118277)

It looks like at least the timeouts are a result of a popup not opening when it should be. Similar to bug 1145199. I don't know why the re-enabling of that test would have caused this to start happening, though it does seem to open the context menu itself, so maybe that's an issue? It also closes it properly and waits for the closure to finish, though, so I don't understand what the issue is. Unless the removeCurrentTab thing maybe animates and interferes here?
Test disabled on Linux. This is failing at a nearly once per push rate, which is simply too high.
https://hg.mozilla.org/integration/mozilla-inbound/rev/0a49cffe0147
Whiteboard: [test disabled on linux][leave open]
Shouldn't we have disabled the test which caused this to fail instead of disabling the failing test.
clearing the needinfo here
Flags: needinfo?(enndeakin)
Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3
Component: General → Notifications and Alerts
Product: Firefox → Toolkit
Severity: normal → S3
Component: Notifications and Alerts → Popup Blocker
You need to log in before you can comment on or make changes to this bug.