1568975 - Restoring session after using RDM will lead to inconsistent state (URL bar != actual page)

Status: NEW

(DevTools :: Responsive Design Mode, defect, P3)

Description

[Beth Rennie [:beth] (she/her)]

STR:

1. Go to example.com
2. Enter RDM
3. Navigate to another page (e.g., google.com)
4. Leave RDM
5. Force-close the browser
6. Start the browser
7. Restore session

ER:
The URL bar should contain the correct URL, not example.com

AR:
You will be at the second page (google.com in this example) but the URL bar will contain example.com

Comment 1

[Beth Rennie [:beth] (she/her)]

After further investigation, on beta the wrong page will be restored (e.g., example.com) but the URLbar will contain the matching URL (example.com). This is really only an issue on Nightly.

Comment 2

[Beth Rennie [:beth] (she/her)]

Upon further further investigation, this is not because of bug 1510569. (I had just assumed it was because this seemed similar to the other RDM bug (bug 1562821)).

:alchen, you've been working on session restore lately. Have you seen this? Do you know if this is because of your work?

Comment 3

[Alphan Chen [:alchen]]

I will be off until next Thursday.
From comment 1, it is an issue happened on Nightly.
Request a regression-window-wanted first.

Comment 4

[Hsin-Yi Tsai (she/her) [:hsinyi]]

This is the regression window I got - https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=50fec259d5a66e44a087253fa5f9f805d32e8ef3&tochange=ad05396bfeed949a112401d661c5d943f775f761

Note that sometimes after the Step 4 (Leave RDM) in STR in comment 0 , I saw the URL became "example.com" while I stayed at the google.com page, even without force-quit and restore.

Comment 5

[Beth Rennie [:beth] (she/her)]

:hsinyi, the patch for bug 1562821 fixes that issue but not the session restore bit.

Comment 6

[Cristian Fogel [:cfogel]]

Updating the regression keywords based on comment 4.

Comment 7

[:Gijs (out for now; he/him)]

Can you mark up which branches are affected here? Is it just 70, and if so, any chance we'll fix this in 70 as well?

Comment 8

[Beth Rennie [:beth] (she/her)]

This only affects 70.

Comment 9

[Liz Henry (:lizzard) (relman/hg->git project)]

Happy to take a patch for 70 but marking fix-optional to remove this from our weekly triage.