Closed Bug 1266475 Opened 8 years ago Closed 8 years ago

Session restore's remoteness switches inadvertently is still switching to about:blank inbetween loads

Categories

(Firefox :: Session Restore, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 48
Tracking Status
firefox48 --- fixed

People

(Reporter: Gijs, Assigned: Gijs)

References

Details

Attachments

(1 file)

As per summary. This causes flickering of the URL in the URL bar when you e.g.:

1. load yahoo.com
2. load about:preferences
3. hit the back button

because we end up clearing the URL bar for about:blank in the middle somewhere.

The comments in ContentRestore.jsm suggest this is not on purpose, and while the flickering is fairly subtle in the current implementation because of quirks in how userTypedClear and userTypedValue behave, it is worse when you try to make those things more sane, as I'm trying to do in bug 1241085. :-)

The good news is that this problem is independently fixable, and I have an automated test for it. Patch incoming.
(I could be convinced to stick the test in a sessionstore directory instead... I guess it doesn't directly verify anything about the location bar. :-\ )
Attachment #8743953 - Flags: review?(mconley) → review+
Comment on attachment 8743953 [details]
MozReview Request: Bug 1266475 - don't re-load about:blank when we're remoteness-switching anyway, r?mconley

https://reviewboard.mozilla.org/r/48197/#review44891

Thanks!

::: browser/base/content/test/urlbar/browser_urlbar_remoteness_switch.js:2
(Diff revision 1)
> +"use strict";
> +

Mind adding a quick docstring describing what this test is doing / testing?
https://hg.mozilla.org/mozilla-central/rev/6d26d2099e1a
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
You need to log in before you can comment on or make changes to this bug.