Use Windows & Mac system API for geolocation

VERIFIED FIXED in Firefox -esr60

Status

()

defect
VERIFIED FIXED
7 months ago
4 months ago

People

(Reporter: sylvestre, Assigned: sylvestre)

Tracking

Trunk
mozilla65
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox-esr6066+ verified, firefox64 wontfix, firefox65 verified)

Details

Attachments

(1 attachment)

The title says all
Assignee: nobody → sledru
We also need to confirm that the fallback from the OS location provider to GLS works correctly and is not too long of a delay.

I will be talking with the Data Science team about running an A/B experiment to compare the quality of the OS location providers' and GLS's results.
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6de1ea2f0c0f
Use Windows & Mac system API for geolocation r=jdm
https://hg.mozilla.org/mozilla-central/rev/6de1ea2f0c0f
Status: NEW → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
QA is planning to test this change using the regular Geolocation test suite. Would that suffice or is there something in particular that should be looked at?
Flags: needinfo?(sledru)
Should be good.
Please try with different OS and versions :)
Thanks!
Flags: needinfo?(sledru)
Blocks: 1513186
Flags: qe-verify+

Emil, has it been verified yet? Thanks

Flags: needinfo?(emil.ghitta)

(In reply to Sylvestre Ledru [:sylvestre] from comment #7)

Emil, has it been verified yet? Thanks

Sorry for not updating things here sooner. Yes, Geolocation has been covered as part of the 65.0b7 sign off - Smoke and Regression Testing revealed no issues following this change.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
Flags: needinfo?(emil.ghitta)

No worries Andrei. thanks!

Depends on: 1527748

Comment on attachment 9029595 [details]
Bug 1512161 - Use Windows & Mac system API for geolocation r?jdm

ESR Uplift Approval Request

If this is not a sec:{high,crit} bug, please state case for ESR consideration

Not a security issue. We want to limit the traffic on the key by leveraging the system API on Windows & Mac

User impact if declined

Should be transparent for the user, this is for Mozilla

Fix Landed on Version

65

Risk to taking this patch

Low

Why is the change risky/not risky? (and alternatives if risky)

We shipped 65 with this change + we pushed an experiment changing that

String or UUID changes made by this patch

Attachment #9029595 - Flags: approval-mozilla-esr60?
Severity: normal → enhancement
Flags: qe-verify+

Comment on attachment 9029595 [details]
Bug 1512161 - Use Windows & Mac system API for geolocation r?jdm

Other than the one issue with the Windows key which we've already remedied, I'm not aware of any negative fallout from the recent Geolocation changes shipped in Fx65 and enabled for ESR60 via experiment. Let's go ahead and land this for 60.6.0 so that we're consistent across all releases and able to reduce our usage of the Google APIs.

Attachment #9029595 - Flags: approval-mozilla-esr60? → approval-mozilla-esr60+

Andrei, it would be great if this could be verified
Thanks
Note that it isnt always enabled by default on windows

Flags: needinfo?(andrei.vaida)
Whiteboard: [qa-triaged]

Verified geolocation on the latest 60.6esr build from taskcluster, no issues were found.

For 65 RC the geolocation tests were covered during the 65.0.2-build1 validation, no issues were encountered.

Flags: qe-verify+
Flags: needinfo?(andrei.vaida)
QA Whiteboard: [qa-triaged]
Whiteboard: [qa-triaged]
Depends on: 1534611
You need to log in before you can comment on or make changes to this bug.