Closed
Bug 1244991
Opened 8 years ago
Closed 8 years ago
Intermittent e10s failure browser_backgroundTab.js | panel shouldn't have shown from a background tab
Categories
(Firefox :: Tours, defect)
Firefox
Tours
Tracking
()
People
(Reporter: MattN, Assigned: mikedeboer)
References
(Blocks 2 open bugs)
Details
(Keywords: intermittent-failure, Whiteboard: [test disabled])
Attachments
(1 file)
3.01 KB,
patch
|
MattN
:
review+
|
Details | Diff | Splinter Review |
e10s on 10.6 and linux32 In test test_info_target_callback INFO - 154 INFO Entering test test_info_target_callback INFO - 155 INFO TEST-UNEXPECTED-FAIL | browser/components/uitour/test/browser_UITour3.js | Event did not happen within 5 seconds. - INFO - Stack trace: INFO - chrome://mochitests/content/browser/browser/components/uitour/test/head.js:add_UITour_task/genFun/</</</funcPromise<:388 INFO - resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:Handler.prototype.process:940 INFO - resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:this.PromiseWalker.walkerLoop:816 INFO - 156 INFO == Done test, doing shared checks before teardown ==
Reporter | ||
Updated•8 years ago
|
Whiteboard: [test disabled]
Comment 2•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b421e073d61e
Updated•8 years ago
|
Blocks: e10s-tests
tracking-e10s:
--- → +
Comment hidden (Intermittent Failures Robot) |
Reporter | ||
Comment 4•8 years ago
|
||
I think bug 1248849 should fix the bug in showInfoPromise that caused this. Try push with chaos mode enabled to test this theory: https://treeherder.mozilla.org/#/jobs?repo=try&revision=a77b3c742b99
Depends on: 1248849
Comment hidden (Intermittent Failures Robot) |
Comment 6•8 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/03dc02bff03d
status-firefox45:
--- → wontfix
status-firefox46:
--- → disabled
status-firefox-esr45:
--- → unaffected
Reporter | ||
Comment 7•8 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM] from comment #6) Note that this may already be fixed by bug 1248849 so we could have tried uplifting that instead…
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → mdeboer
Status: NEW → ASSIGNED
Iteration: --- → 48.2 - Apr 4
Points: --- → 3
Flags: qe-verify-
Flags: firefox-backlog+
Assignee | ||
Comment 9•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=a620dd92a4b9
Assignee | ||
Comment 10•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=bd951fb410ae
Assignee | ||
Comment 11•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=aff438916159
Assignee | ||
Comment 12•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=14fc546e6f59
Assignee | ||
Comment 13•8 years ago
|
||
Since we don't have absolute control over the sequence order of all the various messages that are sent between the content and the main process, we need to add a gate to make sure we wait long enough before we continue to test showPopup. The green try run can be found in comment 12.
Attachment #8736353 -
Flags: review?(MattN+bmo)
Assignee | ||
Comment 14•8 years ago
|
||
IMO, the mix-'n-match of synchronous and asynchronous APIs in UITour is something that needs to be addressed at some point in the client library. But that might be hard if we have to be backwards compatible with that AP surface.
Assignee | ||
Comment 15•8 years ago
|
||
*API
Assignee | ||
Updated•8 years ago
|
Keywords: leave-open
Reporter | ||
Comment 16•8 years ago
|
||
Comment on attachment 8736353 [details] [diff] [review] Patch v1: re-enable browser_backgroundTab.js UITour test Review of attachment 8736353 [details] [diff] [review]: ----------------------------------------------------------------- Thanks. Could you make the commit message more specific now (mentioning visibility / async tab switching, etc.)?
Attachment #8736353 -
Flags: review?(MattN+bmo) → review+
Assignee | ||
Comment 17•8 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/43ee605bf4666503c19e2b5a27972e8cd4a307aa Bug 1244991: re-enable browser_backgroundTab.js UITour test and to make sure it doesn't fail on slower hardware, wait for the visibilitychange event explicitly since it may occur later than the tab open and switch. r=MattN
Comment 18•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/43ee605bf466
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
You need to log in
before you can comment on or make changes to this bug.
Description
•