Closed Bug 1508088 Opened Last year Closed Last year
Don't start new recordings when navigating from blank tab
The attached patch has fixes to get the browser_dbg_rr_console_warp-01.js mochitest passing reliably on try: https://hg.mozilla.org/try/rev/6adbd49fa6745cfe335f50edc581466be22e19f2 This has a couple fixes: - The failures seemed to start happening a lot after bug 1489651 landed. While I don't think there is anything wrong with the patch in that bug in and of itself, it may have exposed timing issues that we did not run into previously. This patch avoids spawning new recordings when navigating in the tests, by watching for navigations from about:blank tabs to somewhere else. Since about:blank tabs don't have any meaningful history this is something that is good to do regardless for efficiency. - Unregister the onConsoleMessage callback when destroying the web console actor. If this hook is called after the actor is destroyed then the test fails to shutdown correctly. I saw this problem while testing locally and I'm not sure if it started cropping up after other changes made while the console warping test was disabled, or if it is related to the failures that caused the test to be disabled.
Attachment #9025901 - Flags: review?(jlaster)
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/0892fe15bc04 Don't start new recording when navigating from blank tab, r=jlast.
You need to log in before you can comment on or make changes to this bug.