Closed
Bug 977455
Opened 12 years ago
Closed 12 years ago
Switch the Tarako over to use the MLS provider
Categories
(Core :: DOM: Geolocation, defect)
Core
DOM: Geolocation
Tracking
()
RESOLVED
FIXED
mozilla30
| Tracking | Status | |
|---|---|---|
| b2g-v1.4 | --- | fixed |
People
(Reporter: dougt, Assigned: dougt)
References
Details
Attachments
(1 file, 1 obsolete file)
|
1.11 KB,
patch
|
fabrice
:
review+
|
Details | Diff | Splinter Review |
Lets switch over some devices to MLS.
Fabrice, we only want to do this on the Tarako (for now). Is there a better way?
Attachment #8382750 -
Flags: feedback?(fabrice)
Comment 1•12 years ago
|
||
Comment on attachment 8382750 [details] [diff] [review]
0003-Switch-geo-wifi-provider-to-MLS.patch
Review of attachment 8382750 [details] [diff] [review]:
-----------------------------------------------------------------
::: b2g/app/b2g.js
@@ +186,4 @@
> pref("privacy.item.syncAccount", true);
>
> // base url for the wifi geolocation network provider
> +pref("geo.provider.use_mls", true);
we'll want that to be false by default, and will just override for devices without gps.
Attachment #8382750 -
Flags: feedback?(fabrice) → feedback+
| Assignee | ||
Comment 2•12 years ago
|
||
so where will we want to do this override?
Flags: needinfo?(fabrice)
Comment 3•12 years ago
|
||
(In reply to Doug Turner (:dougt) from comment #2)
> so where will we want to do this override?
We'll configure that at build time: see https://github.com/mozilla-b2g/gonk-misc/pull/149/files
Flags: needinfo?(fabrice)
| Assignee | ||
Comment 4•12 years ago
|
||
Attachment #8382750 -
Attachment is obsolete: true
Attachment #8384054 -
Flags: review?(fabrice)
| Assignee | ||
Comment 5•12 years ago
|
||
fabrice, this patch just cleans up the main preferences file. We will still need a gaia change.
Updated•12 years ago
|
Attachment #8384054 -
Flags: review?(fabrice) → review+
| Assignee | ||
Comment 6•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/f1c86ed4ccfa
(I am going to file another bug for the gonk change)
Comment 7•12 years ago
|
||
Backed out due to broken Android build:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a761b18588da
https://hg.mozilla.org/integration/mozilla-inbound/rev/01431ff08948
https://hg.mozilla.org/integration/mozilla-inbound/rev/6beb83e91a7d
https://tbpl.mozilla.org/php/getParsedLog.php?id=35459529&tree=Mozilla-Inbound
https://tbpl.mozilla.org/php/getParsedLog.php?id=35460156&tree=Mozilla-Inbound
https://tbpl.mozilla.org/php/getParsedLog.php?id=35460243&tree=Mozilla-Inbound
https://tbpl.mozilla.org/php/getParsedLog.php?id=35459332&tree=Mozilla-Inbound
| Assignee | ||
Comment 8•12 years ago
|
||
Comment 9•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
Updated•12 years ago
|
status-b2g-v1.4:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•