Closed
Bug 963592
Opened 11 years ago
Closed 11 years ago
[b2g] user agent overrides not working correctly
Categories
(Firefox OS Graveyard :: Gaia::Browser, defect)
Tracking
(blocking-b2g:1.3+, firefox27 wontfix, firefox28 fixed, firefox29 fixed, b2g-v1.2 affected, b2g-v1.3 fixed, b2g-v1.4 fixed)
People
(Reporter: bkelly, Assigned: jchen)
References
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
patch
|
fabrice
:
review+
|
Details | Diff | Splinter Review |
Recently I was trying to add a user agent override for a site in order to demonstrate to the developer that it would work on fxos. It appears, though, that maybe the user agent overrides are not working correctly.
To see the problem:
1) Append the following in gecko's b2g/app/ua-updates.json.in:
"whatsmyuseragent.com": "\\(Mobile#(Android; Mobile"
2) Flash gecko on the device.
3) Run make reset-gaia to force the immediate use of the ua-updates.json file.
4) Navigate to whatsmyuseragent.com in the device browser.
When I do this I see:
Your user Agent String is:
\(Mobile(Android; Mobile
It seems we're not actually applied a regex style substitution from this file like we did in v1.1. Instead the contents of ua-updates.json entries simply replace the standard user agent.
Nom'ing for 1.3 since this seems like a serious problem.
Reporter | ||
Updated•11 years ago
|
OS: Windows 8.1 → Gonk (Firefox OS)
Hardware: x86_64 → ARM
Reporter | ||
Comment 1•11 years ago
|
||
Jim, do you have any idea what is going on here?
Blocks: 916205
Flags: needinfo?(nchen)
Assignee | ||
Comment 2•11 years ago
|
||
Looks like regex replacement is not being done for updates from ua-update.json. Should be a simple fix.
Assignee: nobody → nchen
Status: NEW → ASSIGNED
Flags: needinfo?(nchen)
Reporter | ||
Comment 3•11 years ago
|
||
By the way, I saw this on a v1.2 keon as well.
status-b2g-v1.2:
--- → affected
status-b2g-v1.3:
--- → affected
Assignee | ||
Comment 4•11 years ago
|
||
This patch moves override replacement to its own function and calls the function for each override in the updated overrides list. The patch also adds a test for partial overrides.
Attachment #8366087 -
Flags: review?(fabrice)
Updated•11 years ago
|
blocking-b2g: 1.3? → 1.3+
Updated•11 years ago
|
Attachment #8366087 -
Flags: review?(fabrice) → review+
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Comment 5•11 years ago
|
||
Flags: in-testsuite+
Keywords: checkin-needed
Comment 6•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.3 C3/1.4 S3(31jan)
Comment 7•11 years ago
|
||
status-b2g-v1.4:
--- → fixed
status-firefox27:
--- → wontfix
status-firefox28:
--- → fixed
status-firefox29:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•