New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 691835 link

Starred by 2 users
Status: Duplicate
Merged: issue 671580
Owner:
tedc...@chromium.org
Closed: Feb 2017
Cc:
█ aelias@chromium.org
█ k...@chromium.org
creis@chromium.org
█ dfalcant...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

"Request Desktop Site" affects history navigation entries

Project Member Reported by liaoyuke@chromium.org, Feb 14 2017 
Chrome Version: 56.0.2924.87 (Official Build) (32-bit)
OS: Android 7.1.1; Nexus 6P Build/N4F26J

What steps will reproduce the problem?
(1) go to www.medium.com
(2) then go to www.amazon.com
(3) open popup menu and check "Request Desktop Site"
(4) press "go back" button to go back to previous page (www.medium.com)

What is the expected result?
www.medium.com should be loaded using mobile user agent.

What happens instead?
www.medium.com is loaded using desktop user agent.

Please use labels and text to provide additional information.

For graphics-related bugs, please copy/paste the contents of the about:gpu
page at the end of this report.

Comment 1 by creis@chromium.org, Feb 16 2017

Cc: creis@chromium.org k...@chromium.org
Components: UI>Browser>Navigation
Owner: tedc...@chromium.org
tedchoc@ or ktam@, can you help triage this?  (I don't know RDS very well, but my understanding is that the state is stored on the NavigationEntry, and that the behavior described would be a bug.)

Comment 2 by tedc...@chromium.org, Feb 16 2017 

@dfalcantara, do you happen to recall the historical behavior of this?

Comment 3 by dfalcant...@chromium.org, Feb 16 2017 

That's why it went to creis@, actually.  I vaguely recalled someone had been landing CLs to make RDS info propagate across frame navigations or something similar (/me makes hand-wavy motions).  Figured it might be related, but I might be mistaken.

Comment 4 by creis@chromium.org, Feb 16 2017

Cc: aelias@chromium.org
It's possible, though I don't remember it.  There was a recent CL from aelias@ (https://codereview.chromium.org/2619063003/), but I don't think that would be related to this bug.  (I was actually arguing this RDS state shouldn't be in NavigationController on that CL, but there's no need for an urgent change.)  :)

Anyway, looks like it dates back to https://codereview.chromium.org/10170016 and  issue 112923 .

Comment 5 by aelias@chromium.org, Feb 16 2017

Mergedinto: 671580
Status: Duplicate (was: Untriaged)
►

Sign in to add a comment