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)
Firefox
Session Restore
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.
Assignee | ||
Comment 1•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/48197/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/48197/
Attachment #8743953 -
Flags: review?(mconley)
Assignee | ||
Comment 2•8 years ago
|
||
(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. :-\ )
Updated•8 years ago
|
Attachment #8743953 -
Flags: review?(mconley) → review+
Comment 3•8 years ago
|
||
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?
Comment hidden (off-topic) |
Comment 6•8 years ago
|
||
bugherder |
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.
Description
•