1561248 - When returning to previous page, page is scrolled to top

Status: RESOLVED WORKSFORME

(Core :: DOM: Navigation, defect, P3)

Description

[Mirko Brodesser (:mbrodesser-Igalia), offline until April 25h]

STR:

1. Open https://searchfox.org/mozilla-central/source/dom/base/TimeoutHandler.cpp#133-136
2. Click on some link.
3. Press "Alt" and "<-" to return to previous page.

Expected:
page is scrolled to same line as before.

Actual:
page is scrolled to the top.

Happens with current Nighty. Doesn't happen with FF 67.

Comment 1

[Mark Banner (:standard8)]

WFM on latest nightly on Mac (2019-06-24).

Comment 2

[Mirko Brodesser (:mbrodesser-Igalia), offline until April 25h]

Retried with latest Nightly, still doesn't work. Using Ubuntu 18.04.

Comment 3

[Neha Kochar [:neha]]

works for me on Mac using latest nightly 70.0a1 (2019-08-07) (64-bit).
Nika, do you see this on ubuntu?

Comment 4

[Nika Layzell [:nika] (ni? for response)]

By default this works for me on my local install as well.

However, I believe I know what the source of the issue is. The issue only reproduces if the URL being navigated back to is not in the bfcache. This can happen if, for example, there is memory pressure at the time.

You can reproduce this more reliably with the following STR:

1. Open https://searchfox.org/mozilla-central/source/dom/base/TimeoutHandler.cpp#133-136
2. In the DevTools console, type window.onbeforeunload = () => {}, and press enter
3. Click on the "HG Web" link
4. Press "Alt" and "<-" to return to previous page.

The page will incorrectly scroll to the top.

Comment 5

[Mirko Brodesser (:mbrodesser-Igalia), offline until April 25h]

For power-uses of searchfox, this is pretty annoying. On Ubuntu 18.04 this problem always occurs. It does work on Chrome though.

Comment 6

[luc]

Same here, FF 92 Windows.
Happens since ~ version 89

Comment 7

[Mirko Brodesser (:mbrodesser-Igalia), offline until April 25h]

On my machine (Linux) this works nowadays with FF 92 and current Nightly.

Comment 8

[Dennis Schubert [:denschub]]

This seems to work now.