1352850 - Switch to Google for Geolocation services in all builds

Status: RESOLVED FIXED

(SeaMonkey :: Preferences, enhancement)

Description

[rsx11m]

We are currently using the Mozilla service for Geolocation, but only in beta/aurora/trunk builds. Google is used for releases only. This bug should consistently use the Google service for all builds.

Comment 1

[rsx11m]

Hmm, apparently we are using https://location.services.mozilla.com/v1/geolocate?key=%MOZILLA_API_KEY% for geo.wifi.uri in /all/ builds; the distinction was made in its firefox.js counterpart, bug 1216662.

Comment 2

[rsx11m]

Ok, makes sense now, I'll check what those other prefs are doing (likely "false" by default then).

Comment 3

[rsx11m]

Attachment: Proposed pref changes and additions

As discussed in the morning, this switches geo.wifi.uri from the Mozilla to the Google server, assuming the respective API key being present.

As for the other three variables, those are platform-provided geolocation services. Firefox has switched them off by default for release builds as some of them are apparently not yet stable. These services are:

 - On MacOSX, CoreLocation (which is actually part of the iOS SDK)
 - On Windows 8+, its own MS-specific geolocation service
 - On Linux, the GPS daemon (which may or may not be running)

For consistency (and simplification), I'll leave them false for all build types - interested users can always turn they on if they desire to test those.

Comment 4

[Frank-Rainer Grahl (:frg)]

Comment on attachment 8853829 [details] [diff] [review]
Proposed pref changes and additions

Looks good to me. Stealing review. 

Should imho go into 2.49 so you should ask for approval.

Comment 5

[rsx11m]

Thanks, pushed as https://hg.mozilla.org/comm-central/rev/2e7618a2932d

Comment 6

[rsx11m]

Comment on attachment 8853829 [details] [diff] [review]
Proposed pref changes and additions

No point keeping the comm-aurora request around...

Comment 7

[rsx11m]

Comment on attachment 8853829 [details] [diff] [review]
Proposed pref changes and additions

Ewong, unless you want this for 2.48 already, the comm-release request is actually for 2.49.x and thus comm-esr52 (we don't have separate flags yet for SeaMonkey).

Comment 8

[Edmund Wong (:ewong)]

(In reply to rsx11m from comment #7)
> Comment on attachment 8853829 [details] [diff] [review]
> Proposed pref changes and additions
> 
> Ewong, unless you want this for 2.48 already, the comm-release request is
> actually for 2.49.x and thus comm-esr52 (we don't have separate flags yet
> for SeaMonkey).

We'll just do this for 2.48. I'm hesitant in introducing a change to something
that wasn't in beta.

Edmund

Comment 9

[rsx11m]

Comment on attachment 8853829 [details] [diff] [review]
Proposed pref changes and additions

Switching approval-comm-release? to approval-comm-esr52? now that we have it.

Comment 10

[Ian Neal]

Comment on attachment 8853829 [details] [diff] [review]
Proposed pref changes and additions

a=me for c-b and c-esr52

Comment 11

[rsx11m]

Thanks, pushed to branches:
https://hg.mozilla.org/releases/comm-beta/rev/52af4abc69cc
https://hg.mozilla.org/releases/comm-esr52/rev/0289df8911d5