Closed
Bug 503202
Opened 15 years ago
Closed 15 years ago
Change Windows CE user agent string
Categories
(Core :: Networking, defect)
Tracking
()
VERIFIED
FIXED
mozilla1.9.2a1
People
(Reporter: Dolske, Assigned: Dolske)
References
Details
(Whiteboard: [nv])
Attachments
(1 file, 1 obsolete file)
609 bytes,
patch
|
vlad
:
review+
|
Details | Diff | Splinter Review |
Some sites sniff for "Windows CE" in the User-Agent header, and serve back a stripped-down "mobile" version of the site. For example, CNN.com does this (in fact redirecting you to m.cnn.com). We really don't want this happening for Firefox on Windows CE (we can render the full content, and on some devices in fact have a normal high-res screen). Vlad suggests just changing it to "WinCE".
Assignee | ||
Updated•15 years ago
|
Whiteboard: [nv]
Assignee | ||
Updated•15 years ago
|
Component: General → Networking
Product: Firefox → Core
QA Contact: general → networking
Assignee | ||
Comment 1•15 years ago
|
||
Assignee: nobody → dolske
Attachment #387570 -
Flags: review?(vladimir)
Comment 2•15 years ago
|
||
I like dolske's other proposal "Windows 6.0 (CE)"
Let's go with "WindowsCE" (no space).
Assignee | ||
Updated•15 years ago
|
Summary: Change "Windows CE" to "WinCE" in user agent string → Change Windows CE user agent string
Assignee | ||
Comment 5•15 years ago
|
||
Attachment #387570 -
Attachment is obsolete: true
Attachment #387703 -
Flags: review?(vladimir)
Attachment #387570 -
Flags: review?(vladimir)
Assignee | ||
Updated•15 years ago
|
Attachment #387703 -
Attachment description: Patch v.1 → Patch v.2
Assignee | ||
Comment 6•15 years ago
|
||
Comment on attachment 387703 [details] [diff] [review] Patch v.2 Oops, this is v.2 ("WindowsCE, no space).
Comment 7•15 years ago
|
||
What will navigator.userAgent return? IIRC that's controlled by prefs (at least in FF).
Attachment #387703 -
Flags: review?(vladimir) → review+
Comment on attachment 387703 [details] [diff] [review] Patch v.2 Looks good, please land away
Assignee | ||
Comment 9•15 years ago
|
||
Pushed http://hg.mozilla.org/mozilla-central/rev/4b3fc86f893e
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2a1
Would you mind adding WindowsCE to https://developer.mozilla.org/En/User_Agent_Strings_Reference ?
Comment 11•15 years ago
|
||
So this doesn't always solve the problem. It does a much better job than before but some redirectors seem to be a little smarter. See http://tw.yahoo.com/ (not that we're even able to render the characters on that page)
Comment 12•15 years ago
|
||
IE Mobile 6 has two modes: Desktop and Mobile It changes the UA string depending on the mode. IEM6, Desktop: Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1) IEM6, Mobile: Mozilla/4.0 (compatible; MSIE 6.0; Windows CE; IEMobile 8.12; MSIEMobile 6.0) http://blogs.msdn.com/windowsmobile/archive/2008/11/11/internet-explorer-mobile-6.aspx http://nino.net/blog/internet-explorer-mobile-6/ Anyone think we should just do the same?
I don't; the "Desktop" version just makes it pretend to be Desktop IE. We don't want to pretend to be the desktop I don't think. Fennec should just set browser.useragent.extra.mobile or something to say "FennecMobile" or whatever to add it to the user agent string.
Comment 14•15 years ago
|
||
Verified fix on Tegra: Mozilla/5.0 (Windows; U; WindowsCE 6.0; en-US; rv:1.9.2a2pre) Gecko/20090818 Namaroka/3.6a2pre. also Verified fix on Winmo: Mozilla/5.0 (Windows; U; WindowsCE 5.2; en-US; rv:1.9.3a1pre) Gecko/20090817 Fennec/1.0a3pre.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•