1698192 - Accessing currentURI of a lazy browser while the browser is swapped corrupts the cached currentURI

Status: RESOLVED FIXED

(Firefox :: Tabbed Browser, defect)

Description

[Rob Wu [:robwu]]

There was a regression in extensions (bug 1694699) due to an incorrect value for currentURI. I have fixed the trigger in bug 1694699 for that specific test case and fixed another trigger in bug 1695346. But these fixes didn't touch the root cause in the tabbrowser/session restore code. I'll attach the fix for the root cause along with a regression test to this bug.

See https://bugzilla.mozilla.org/show_bug.cgi?id=1695346#c1 for a concise explanation of the bug (and the fix).

Comment 1

[Rob Wu [:robwu]]

Attachment: Bug 1698192 - Use correct currentURI during swapping of lazy browsers

Re-order calls in SessionStore.jsm to invoke the side-effect-free logic
before triggering observable events.

Clear (likely) stale value of the cached currentURI when lazy browsers
are swapped.

And while I was at it, I also fixed a bug that caused a lazy browser to
eagerly be unlazified because the unneeded mute() call on a lazy browser
unlazifies it (discovered when I added complete test coverage for this
change).

Comment 2

[Rob Wu [:robwu]]

Dao, do you have time to review this or should I look for a different reviewer?

The fix itself is just one line of code + moving existing code around.

Comment 3

[Dão Gottwald [:dao]]

Done, sorry for the delay.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6820a8d06219
Use correct currentURI during swapping of lazy browsers r=dao

Comment 5

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/6820a8d06219