Closed Bug 1333516 Opened 7 years ago Closed 7 years ago

geolocation not working in Firefox 51 release

Categories

(Core :: DOM: Geolocation, defect)

Product:
Core
Component:
DOM: Geolocation
Version:
51 Branch
Platform:
x86_64
Windows 10
Type:
defect
Priority:
Not set
Severity:
major

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla54
Tracking Flags:
Tracking Status
firefox51 blocking verified
firefox52 + verified
firefox53 --- verified
firefox54 --- verified

People

(Reporter: asa, Assigned: glandium)

References

Details

(Keywords: regression)

Attachments

(1 file)

Bug 1333516 - Fix windows path for API keys after bug 1294585.
59 bytes, text/x-review-board-request
gps
: review+
ritu
: approval-mozilla-aurora+
ritu
: approval-mozilla-beta+
ritu
: approval-mozilla-release+
Details 
Geolocation doesn't work at Google Maps or any other site I've tested on Windows Firefox 51 release build.

Steps:

1. visit https://www.google.com/maps
2. click the location button on the map
3. say yes to the confirmation dialog

Results

Several seconds pass and then "Your location could not be determined."

Also tested at https://www.mozilla.org/en-US/firefox/geolocation/ and at http://www.w3schools.com/HTML/tryit.asp?filename=tryhtml5_geolocation_error and both fail as well.

Tested on Mac and everything seems to work.
API keys are missing in Windows builds (tested 32 bit build). From nsURLFormatter.js in the build:
>    MOZILLA_API_KEY:  () => "no-mozilla-api-key",
>    GOOGLE_API_KEY:   () => "no-google-api-key",
>    BING_API_CLIENTID:() => "no-bing-api-clientid",
>    BING_API_KEY:     () => "no-bing-api-key",
This appears to be a regression in 51. I can repro the problem in 51, but not in 50.1. I can't reproduce in Aurora 52 or Nightly 54, but those pre-release channels use Mozilla's location service instead of Microsoft's.
status-firefox51: --- → affected
status-firefox52: --- → ?
status-firefox53: --- → unaffected
status-firefox54: --- → unaffected
Last good: 50.0b11
First bad: 51.0b1

jlund, are you aware of changes in releng which could cause this?
Flags: needinfo?(jlund)
Is this also affecting Firefox for Android?
Kan-Ru/Josh/Garvan, can any of you investigate? I can reproduce on my laptop if you'd like to try locally, Josh (or Garvan - I'll buy you a beer ;).
Flags: needinfo?(kchen)
Flags: needinfo?(josh)
Flags: needinfo?(garvankeeley+bmo)
(In reply to Sebastian Hengst [:aryx][:archaeopteryx] (needinfo on intermittent or backout) from comment #3)
> Last good: 50.0b11
> First bad: 51.0b1
> 
> jlund, are you aware of changes in releng which could cause this?

I'm not aware of anything releng related that would affect this. Not ruling it out, just saying it is unlikely. Releng focuses on the engineering of 'how to ship code', rather than the management of 'what code is shipped'.

release-mgmt + devs would have a better idea of what exactly landed on 51.0b1; likely something that got migrated from mozilla-aurora->mozilla-beta. I would bet this affected aurora/developer-edition when 51 got migrated from central/nightly.
Flags: needinfo?(jlund)
Don't have 51 release yet, but tried 51 Beta and noticed the same issue that Asa described for Google maps. However, once I moved onto the firefox geolocation test page, it worked right away, went back to Google maps and it ended up finding my location there as well.
^ this was meant for Android, sorry.
Looks like it's windows specific. Tried on linux64 and win7 (64bit), the latter didn't work.

Mark, do you happen to know if we changed anything related to https://dxr.mozilla.org/build-central/source/relabs-puppet/modules/slave_secrets/manifests/mozilla_geoloc_api_keys.pp for windows recently?

The release log https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-release-win64/1484771846/mozilla-release-win64-bm73-build1-build38.txt.gz passes the path as:

12:50:40     INFO -    --with-mozilla-api-keyfile=/c/builds/mozilla-desktop-geoloc-api.key
Flags: needinfo?(mcornmesser)
jcristau just mentioned bug 1294585 in IRC, can be related
(In reply to Rail Aliiev [:rail] ⌚️ET from comment #10)
> Looks like it's windows specific. Tried on linux64 and win7 (64bit), the
> latter didn't work.
> 
> Mark, do you happen to know if we changed anything related to
> https://dxr.mozilla.org/build-central/source/relabs-puppet/modules/
> slave_secrets/manifests/mozilla_geoloc_api_keys.pp for windows recently?
> 
> The release log
> https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-release-
> win64/1484771846/mozilla-release-win64-bm73-build1-build38.txt.gz passes the
> path as:
> 
> 12:50:40     INFO -   
> --with-mozilla-api-keyfile=/c/builds/mozilla-desktop-geoloc-api.key

I am not seeing any changes made to it. I just verified that they were on the 2008 instances and have what appears to be usable permissions.
Flags: needinfo?(mcornmesser)
Adding Mike per comment 7.
Flags: needinfo?(mh+mozilla)
> Kan-Ru/Josh/Garvan, can any of you investigate? I can reproduce on my laptop
> if you'd like to try locally, Josh (or Garvan - I'll buy you a beer ;).

If it helps, I can take a look in a few hours. I just got a Windows 10 gaming box, I *suppose* I can use it for other things. (In reply to Andrew Overholt [:overholt] from comment #5).
Thanks, Garvan. It looks like it's something that gets substituted at build time on the build machines so not likely something you can help with. That beer is still on offer :)
Flags: needinfo?(kchen)
Flags: needinfo?(josh)
Flags: needinfo?(garvankeeley+bmo)
I don't think this is related but I am the "owner" of the GLS account (especially since Doug left us) :)
(fwiw, it's likely that safebrowsing is also broken due to the missing keys)
From https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-release-win64/1484771846/mozilla-release-win64-bm73-build1-build38.txt.gz:

12:50:41     INFO -  checking for zip... c:/mozilla-build/info-zip/zip.EXE
12:50:41     INFO -  checking for the Mozilla API key... no
12:50:41     INFO -  checking for the Google API key... no
12:50:41     INFO -  checking for the Bing API key... no
12:50:41     INFO -  checking for the Adjust SDK key... no

from https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-release-linux/1484771846/mozilla-release-linux-bm77-build1-build45.txt.gz:

12:52:32     INFO -  checking for zip... /usr/bin/zip
12:52:32     INFO -  checking for the Mozilla API key... yes
12:52:32     INFO -  checking for the Google API key... yes
12:52:32     INFO -  checking for the Bing API key... no
12:52:32     INFO -  checking for the Adjust SDK key... no

These are coming from https://dxr.mozilla.org/mozilla-central/source/toolkit/moz.configure#551-563 

Summoning gps, :) he may have some insights on how those calls are converted into filenames.
Flags: needinfo?(gps)
(In reply to Rail Aliiev [:rail] ⌚️ET from comment #18)
> 12:50:41     INFO -  checking for the Mozilla API key... no
> 12:50:41     INFO -  checking for the Google API key... no

Can we update the configure to fail the build when the beta or release branding are being used but cannot find api keys?
(In reply to Sylvestre Ledru [:sylvestre] from comment #21)
> (In reply to Rail Aliiev [:rail] ⌚️ET from comment #18)
> > 12:50:41     INFO -  checking for the Mozilla API key... no
> > 12:50:41     INFO -  checking for the Google API key... no
> 
> Can we update the configure to fail the build when the beta or release
> branding are being used but cannot find api keys?

That's bug 1256730 and is waiting for bug 1293643 to be fixed... but now I realize maybe the actual problem is what's causing *this* bug, being that the path is an msys path and configure expects a windows path.
Flags: needinfo?(mh+mozilla)
Assignee: nobody → mh+mozilla
glandium provided an answer. I'll be around for the next few hours to review any changes. Just ping me on IRC to get my attention.
Flags: needinfo?(gps)
Thanks!
Comment on attachment 8830052 [details]
Bug 1333516 - Fix windows path for API keys after bug 1294585.

https://reviewboard.mozilla.org/r/106976/#review108028
Attachment #8830052 - Flags: review?(gps) → review+
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/autoland/rev/56147a136a8c
Fix windows path for API keys after bug 1294585. r=gps
I have a try build with this patch + bug 1256730 running, and I also want to see what happens on that autoland push before pushing to other branches.
The TC builds are good. Waiting for buildbot builds. As for bug 1256730, it looks like only cross-osx builds are a problem at the moment.
Hi GPS, please nominate patches for uplift to Aurora53, Beta52 and Release51 when ready. Thanks! We are hoping to gtb 51.0.1 in the next couple of hours, mainly driven by this issue.
Flags: needinfo?(gps)
Redirecting needinfo to glandium since the changes are his and he is actively testing things.
Flags: needinfo?(gps) → needinfo?(mh+mozilla)
Comment on attachment 8830052 [details]
Bug 1333516 - Fix windows path for API keys after bug 1294585.

We're good to go, the buildbot builds are good.

Approval Request Comment
[Feature/Bug causing the regression]: regression from bug 1294585
[User impact if declined]: Geolocation doesn't work
[Is this code covered by automated tests?]: no
[Has the fix been verified in Nightly?]: verified on autoland
[Needs manual test from QE? If yes, steps to reproduce]: see comment 0
[List of other uplifts needed for the feature/fix]: we may want to push bug 1256730 along when it's ready to land (which is not the case currently)
[Is the change risky?]: no
[Why is the change risky/not risky?]: The patch is changing a non-working path to a working path in a mozconfig.
[String changes made/needed]: none
Flags: needinfo?(mh+mozilla)
Attachment #8830052 - Flags: approval-mozilla-release?
Attachment #8830052 - Flags: approval-mozilla-beta?
Attachment #8830052 - Flags: approval-mozilla-aurora?
Liz wanted this pushed to release ASAP, so I went ahead and landed it on m-r before the official a+:
https://hg.mozilla.org/releases/mozilla-release/rev/cac8c9e7e0efdb615504f66ede2cffeddca348ee
I'll wait for official approval before landing on the other branches
status-firefox51: affected → fixed
Comment on attachment 8830052 [details]
Bug 1333516 - Fix windows path for API keys after bug 1294585.

We need this landed on 51+
Attachment #8830052 - Flags: approval-mozilla-release?
Attachment #8830052 - Flags: approval-mozilla-release+
Attachment #8830052 - Flags: approval-mozilla-beta?
Attachment #8830052 - Flags: approval-mozilla-beta+
Attachment #8830052 - Flags: approval-mozilla-aurora?
Attachment #8830052 - Flags: approval-mozilla-aurora+
https://hg.mozilla.org/mozilla-central/rev/56147a136a8c
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Flags: qe-verify+
https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-release-win64/1485304068/mozilla-release-win64-bm94-build1-build32.txt.gz looks much better:

16:53:14     INFO -  checking for zip... c:/mozilla-build/info-zip/zip.EXE
16:53:14     INFO -  checking for the Mozilla API key... yes
16:53:14     INFO -  checking for the Google API key... yes
16:53:14     INFO -  checking for the Bing API key... no
16:53:14     INFO -  checking for the Adjust SDK key... no
The issue is verified fixed on 51.0.1-build2 (20170124203848) using Windows 10 x64, Windows 7 x64, Mac OS X 10.11 and Ubuntu 16.04 x64.
status-firefox51: fixed → verified
See Also: → 1333783
I reported bug 1333783 to make sure that we have automation to avoid this situation in the future.
Discussing with Andrei, he confirmed that we tested safe browsing in beta7:
https://wiki.mozilla.org/Releases/Firefox_51/Test_Plan#Beta_7
https://drive.google.com/file/d/0B6_25UfoAoJhUHgtMkVHaVlMQVU/view
and it was working.
See Also: → 1336920
Tested on :
Nightly 54.0a1 (2017-02-09)
Aurora 53.0a2  (2017-02-09)
Beta 52.0b4    (20170206101855)

Verified fixed on Windows 10 , Ubuntu 16.04 and Mac OS X 10.11 , using the links from comment 0: https://www.google.com/maps
https://www.mozilla.org/en-US/firefox/geolocation/ 
http://www.w3schools.com/HTML/tryit.asp?filename=tryhtml5_geolocation_error.
Status: RESOLVED → VERIFIED
status-firefox52: fixed → verified
status-firefox53: fixed → verified
status-firefox54: fixed → verified
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: