1952788 - m.youtube.com - Redirects to homepage when going back after accessing a video from search results

Status: RESOLVED FIXED

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

Description

[Calin Tanase]

Environment:
Operating system: Android 14
Firefox version: Firefox Mobile [phone]

Steps to reproduce:

1. Go to https://m.youtube.com/
2. Dismiss the cookie banner.
3. Search for something.
4. Access a video.
5. Tap on the back button.

Expected Behavior:
Returns to search results page.

Actual Behavior:
Returns to homepage.

Notes:

• Reproduces regardless of the status of ETP
• Reproduces in firefox-nightly, and firefox-release
• Does not reproduce in chrome

Created from https://github.com/webcompat/web-bugs/issues/149622

Comment 1

[Calin Tanase]

Attachment: FF vs Chrome.mp4

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

Since nightly and release are affected, beta will likely be affected too.
For more information, please visit BugBot documentation.

Comment 3

[Dennis Schubert [:denschub][OOO, back Aug 11]]

This might be related to the history API? But that'd be concerning, so I hope this is just a YouTube-introduced issue.

Comment 4

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

Does the issue go away if you disable browser.navigation.requireUserInteraction in about:config?

There's a patch for a recent regression on YouTube bug 1946547, which may be related or shares the same root cause.

Comment 5

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

(In reply to Hsin-Yi Tsai (she/her) [:hsinyi] from comment #4)

Does the issue go away if you disable browser.navigation.requireUserInteraction in about:config?

I confirmed if I disabled browser.navigation.requireUserInteraction, the issue went away.

There's a patch for a recent regression on YouTube bug 1946547, which may be related or shares the same root cause.

Adam, does this share the same root cause as bug 1946547?

Comment 6

[Adam Vandolder [:avandolder]]

No, I don't believe this is related to bug 1946547 - the issue here appears to be that the search results history entry isn't being marked as having been user-interacted, even though it should be. At this point, I'm not clear on why that is happening, however.

Comment 7

[Adam Vandolder [:avandolder]]

Attachment: Bug 1952788 - Fix Document::NotifyUserGestureActivation not properly setting user-interaction state. r?farre,#dom-core

Comment 8

[BugBot [:suhaib / :marco/ :calixte]]

There is an r+ patch which didn't land and no activity in this bug for 2 weeks.
:avandolder, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit BugBot documentation.

Comment 9

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

It's a platform bug IIUC.

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/1d2def4b3da1
https://hg.mozilla.org/integration/autoland/rev/d4d34bd37994
Fix Document::NotifyUserGestureActivation not properly setting user-interaction state. r=farre,dom-core

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/03cabb9da133
https://hg.mozilla.org/integration/autoland/rev/d8cb13769869
Revert "Bug 1952788 - Fix Document::NotifyUserGestureActivation not properly setting user-interaction state. r=farre,dom-core" for causing bc failures on browser_storageAccessPrivilegeAPI.js

Comment 12

[Cristina Horotan [:chorotan]]

Backed out for causing bc failures on browser_storageAccessPrivilegeAPI.js

Backout link

Push with failures

Failure log

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/1cda380e0211
https://hg.mozilla.org/integration/autoland/rev/65ff7d806d1c
Fix Document::NotifyUserGestureActivation not properly setting user-interaction state. r=farre,dom-core

Comment 14

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/65ff7d806d1c