Closed
Bug 941410
Opened 12 years ago
Closed 12 years ago
navigator.platform is '' on b2g linux
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: daleharvey, Assigned: daleharvey)
References
Details
No description provided.
| Assignee | ||
Comment 1•12 years ago
|
||
It is reporting correctly on OSX though
It seems to be getting set in
http://mxr.mozilla.org/mozilla-central/source/netwerk/protocol/http/nsHttpHandler.cpp#669
naively tried changing MOZ_B2G to MOZ_WIDGET_GONK but didnt help, investigating but if anyone wants to steal that would be nice, probably a quick fix for someone who isnt me
| Assignee | ||
Updated•12 years ago
|
Assignee: nobody → dale
| Assignee | ||
Comment 2•12 years ago
|
||
It doesnt look like we go through http://mxr.mozilla.org/mozilla-central/source/netwerk/protocol/http/nsHttpHandler.cpp#688 at all, so not sure why this wouldnt pick up the same navigator.platform as it does on linux
| Assignee | ||
Comment 3•12 years ago
|
||
Looks like its because we set MOZ_UA_OS_AGNOSTIC which purposefully sets navigator.platform to ''
https://bugzilla.mozilla.org/show_bug.cgi?id=777710
So this may be resolved 'wontfix', but if so we need another solution for https://bugzilla.mozilla.org/show_bug.cgi?id=935672
| Assignee | ||
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
Comment 4•12 years ago
|
||
(In reply to Dale Harvey (:daleharvey) from comment #1)
> It is reporting correctly on OSX though
>
Should B2G Mac report '' too? If so we should file a bug and make it so.
Flags: needinfo?(dale)
| Assignee | ||
Comment 5•12 years ago
|
||
Looking at the code and the intention, I think b2g mac should be '' as well, pretty confusing to have people test / depend on it on desktop then see it break on device
Flags: needinfo?(dale)
Comment 6•12 years ago
|
||
(In reply to Dale Harvey (:daleharvey) from comment #5)
> Looking at the code and the intention, I think b2g mac should be '' as well,
> pretty confusing to have people test / depend on it on desktop then see it
> break on device
Filed bug 957909.
You need to log in
before you can comment on or make changes to this bug.
Description
•