Closed Bug 1508088 Opened Last year Closed Last year

Don't start new recordings when navigating from blank tab


(Core :: Web Replay, enhancement)

Not set



Tracking Status
firefox65 --- fixed


(Reporter: bhackett, Assigned: bhackett)


(Blocks 1 open bug)



(1 file)

Attached patch patchSplinter Review
The attached patch has fixes to get the browser_dbg_rr_console_warp-01.js mochitest passing reliably on try:

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)
Attachment #9025901 - Flags: review?(jlaster) → review+
Pushed by
Don't start new recording when navigating from blank tab, r=jlast.
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in before you can comment on or make changes to this bug.