Closed Bug 1237075 Opened 4 years ago Closed 4 years ago

Navigating from 'manual' to 'auto' session history entry should scroll the page

Categories

(Core :: DOM: Navigation, defect)

36 Branch
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla46
Tracking Status
firefox46 --- fixed

People

(Reporter: smaug, Assigned: smaug)

References

Details

Attachments

(1 file)

.
Assignee: nobody → bugs
patch coming (even though the spec doesn't require this behavior, this is up to the UA)
Attached patch patchSplinter Review
In case we're doing history load, we don't want to use scroll restoration state from mOSHE, but from the to-be-loaded SHEntry.
Attachment #8704353 - Flags: review?(jst)
The patch makes Gecko to pass https://bugzilla.mozilla.org/show_bug.cgi?id=1155730#c11
(that test does rely on UA specific behavior, but since we're scrolling in other cases, I think we should scroll in this case too)
Attachment #8704353 - Flags: review?(jst) → review+
Summary: Scrolling from 'manual' to 'auto' session history entry should scroll the page → Navigating from 'manual' to 'auto' session history entry should scroll the page
https://hg.mozilla.org/mozilla-central/rev/7eb46e50210c
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in before you can comment on or make changes to this bug.