Remove m.foo.com -> foo.com RDS hack

RESOLVED FIXED in Firefox 30

Status

()

Firefox for Android
General
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: bnicholson, Assigned: bnicholson)

Tracking

Trunk
Firefox 30
All
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
When using Request Desktop Site, we have a basic algorithm that tries to purge as much as possible from the URL to guarantee that we aren't stuck on a mobile site: http://mxr.mozilla.org/mozilla-central/source/mobile/android/chrome/content/browser.js#2989. This hack is in place to address complaints that other browsers face, which is that RDS doesn't seem to do anything on many mobile sites (because a mobile URL still shows a mobile page, regardless of the UA).

Although it increases the success rate of switching from a mobile to desktop page, it results in the user being moved to the root of that domain (foo.com/a/b/c -> foo.com). Aside from the fact that users are moved to a different page than the one they were viewing, the root domain may not be meant for viewing, resulting in an error page. Additionally, this hack won't work for sites that don't follow the "m." prefix convention; e.g., i.reddit.com or mobile.twitter.com.

While we haven't received many complaints that I know of, this approach is an ugly hack that can break things, so this is more of an add-on territory.
(Assignee)

Comment 1

4 years ago
Created attachment 8376460 [details] [diff] [review]
Remove Request Desktop Site URL rewrite hack
Attachment #8376460 - Flags: review?(mark.finkle)
Attachment #8376460 - Flags: review?(mark.finkle) → review+
https://hg.mozilla.org/mozilla-central/rev/267e352db60c
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 30
You need to log in before you can comment on or make changes to this bug.