Status

Firefox for Android Graveyard
General
VERIFIED FIXED
8 years ago
8 years ago

People

(Reporter: vlad, Assigned: mbrubeck)

Tracking

Details

Attachments

(1 attachment, 2 obsolete attachments)

Right now, the Fennec on Maemo UA looks like:

Mozilla/5.0 (X11; U; Linux armv7l; en-US; rv:1.9.2.4) Gecko/20100515 ...

This is indistinguishable from any other Linux, which I believe was the intent, but the end result is that now sites are treating Gecko + "Linux armv7" = n900.  For example, if you go to facebook, it'll redirect you to m.facebook.com and offer to install a facebook bookmark for your n900.

I suggest that we add "Maemo" to this (so "X11; U; Maemo; Linux armv7l; ..."), and then work on a standard that allows UAs to tell web sites whether they want a mobile optimized version of a site or not.  Something as simple as a header could do it.
(Assignee)

Updated

8 years ago
Assignee: nobody → mbrubeck
Status: NEW → ASSIGNED
OS: Windows 7 → Linux (embedded)
Hardware: x86 → ARM
(Assignee)

Comment 1

8 years ago
Created attachment 450474 [details] [diff] [review]
patch

This changes the OS/CPU string from "Linux armv7l" to "Maemo Linux armv7l" on N900 (and similarly for other Maemo devices).

Note that this will not break existing sites that look for the exact string "Linux armv7l".
Attachment #450474 - Flags: review?(vladimir)

Comment 2

8 years ago
We would need to follow https://developer.mozilla.org/en/User_Agent_Strings_Reference -- we could possibly change Linux to Maemo4 or 5 or whatever
(Assignee)

Comment 3

8 years ago
Yeah, if we want to change the OS/CPU string, then we need to update that spec to make Maemo a separate case from other "Unix" operating systems.

Another possibility is to use "Maemo" instead of "X11" as the Platform string.  X11 is technically accurate but not very useful as a platform identifier for Maemo.  (This is also where we put "Android" in our UA string.)

Or we could include another VendorProductToken like "Maemo/5" in addition to "Fennec/2.0a1pre" (without any spec changes).
Comment on attachment 450474 [details] [diff] [review]
patch

er, this patch a has a lot of non-UA change stuff in it?
(Assignee)

Comment 5

8 years ago
Created attachment 450496 [details] [diff] [review]
patch v2

(In reply to comment #4)
> er, this patch a has a lot of non-UA change stuff in it?

Oops.  Here's the right patch.
Attachment #450474 - Attachment is obsolete: true
Attachment #450496 - Flags: review?(vladimir)
Attachment #450474 - Flags: review?(vladimir)
Attachment #450496 - Flags: review?(vladimir) → review+
Comment on attachment 450496 [details] [diff] [review]
patch v2

Sure, that UA change looks fine to me, though I raelize now that it's in netwerk code.. you may want to get someone else to ok it as well, maybe dwitte or smoeone who's been looking at UA stuff?  Dunno.
(Assignee)

Updated

8 years ago
Attachment #450496 - Flags: review?(cbiesinger)

Updated

8 years ago
Component: Linux/Maemo → General
(Assignee)

Updated

8 years ago
Blocks: 586165

Comment 7

8 years ago
Comment on attachment 450496 [details] [diff] [review]
patch v2

Let's replace X11 with Maemo instead: "Maemo; Linux ..."

r=me with that.
Attachment #450496 - Flags: review?(cbiesinger) → review+
(Assignee)

Comment 8

8 years ago
Created attachment 469264 [details] [diff] [review]
patch v3: Replace "X11" with "Maemo"

Updated as requested in comment 7, carrying the review.
Attachment #450496 - Attachment is obsolete: true
Attachment #469264 - Flags: superreview?(jst)
Attachment #469264 - Flags: review+
(Assignee)

Updated

8 years ago
tracking-fennec: --- → ?

Updated

8 years ago
Attachment #469264 - Flags: superreview?(jst) → superreview+
tracking-fennec: ? → 2.0b1+
(Assignee)

Comment 9

8 years ago
Pushed: http://hg.mozilla.org/mozilla-central/rev/b2bc1dd449b7
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
verified FIXED on build:

Mozilla/5.0 (X11; U; Linux armv71; Nokia N900; en-US; rv:2.0b5pre) Gecko/20100826 Namoroka/4.0b5pre Fennec/2.0a1pre
Status: RESOLVED → VERIFIED

Comment 11

8 years ago
(In reply to comment #10)
> verified FIXED on build:
> 
> Mozilla/5.0 (X11; U; Linux armv71; Nokia N900; en-US; rv:2.0b5pre)
> Gecko/20100826 Namoroka/4.0b5pre Fennec/2.0a1pre

That doesn't make sense. I thought this bug replaced X11 with Maemo, so how can it be verified in a build with a UA that doesn't show this bug as fixed per the summary?
You need to log in before you can comment on or make changes to this bug.