MozReview Request: Bug 1266475 - don't re-load about:blank when we're remoteness-switching anyway, r?mconley
58 bytes, text/x-review-board-request
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.
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)
(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?
You need to log in before you can comment on or make changes to this bug.