Regression from bug 747883; the URL fixup stuff inadvertently treats page reloads as "spurious addition"s to the URL.
Created attachment 631962 [details] [diff] [review] Make page reloads set !sameDocument
Also CC'ing ajuma in case he has any comments.
Noming this as it blocks bug 759678, which in turn could fix bug 721523, which is a blocker.
Not a blocker because bug 721523 is going to be wallpapered over AMO side, aiui. Would like, though (and feel free to approvalnom if appropriate).
Comment on attachment 631962 [details] [diff] [review] Make page reloads set !sameDocument [Approval Request Comment] Bug caused by (feature/regressing bug #): bug 747883 User impact if declined: on page reload, the page might jump around during load, since we don't suspend viewport updates and user interaction. Testing completed (on m-c, etc.): on m-c for a day Risk to taking this patch (and alternatives if risky): mobile-only. potential risk of either regressing bug 747883 or not fully fixing the issues described in "user impact if declined" String or UUID changes made by this patch: none Also note that the patch on bug 759678 will be ineffective for page reloads without this patch.
Comment on attachment 631962 [details] [diff] [review] Make page reloads set !sameDocument Discussed in triage with kats - approved for b7. Go go go!
https://hg.mozilla.org/releases/mozilla-aurora/rev/ef143fb2db67 https://hg.mozilla.org/releases/mozilla-beta/rev/e49fb24c3476 and on the b7 relbranch: https://hg.mozilla.org/releases/mozilla-beta/rev/7e41f8cab7c6
Verified on Firefox 14 beta 7 build 2