Created attachment 353375 [details] [review]
patch solving the bug

Review of attachment 353375 [details] [review]:

Thanks for the patch!
Looks good (only a few styling nitpicks)!

::: src/geoclue.js
@@ +124,3 @@
+        Utils.debug("Updated location: " + location.description
+            + " (" + location.latitude + ", " + location.longitude
+            + ", accuracy = " + location.accuracy + " m)");

I think I prefer the +'s to be at the end of the line when continuing concatenation on the next line, and then have the starting " on each line match up with the " in the beginning "Updated location:..." string.

Created attachment 353378 [details] [review]
updated patch

Review of attachment 353378 [details] [review]:

LGTM!
And it works great!

Pushed (I modified the commit message a bit to follow our convensions, btw).

Thanks! This allowed me to find a bug in geoclue:
https://bugs.freedesktop.org/show_bug.cgi?id=101357
Hopefully gnome-maps will use my GPS location on my laptop, now.

(In reply to Valentin Blot from comment #6)
> Thanks! This allowed me to find a bug in geoclue:
> https://bugs.freedesktop.org/show_bug.cgi?id=101357
> Hopefully gnome-maps will use my GPS location on my laptop, now.

Great!

Btw, there's also the geoclue demo application (probably located under /usr/lib/geoclue-2.0/demos/where-am-i or something similar, might be distro-dependent, and maybe you needed to install a separate package, don't quite remember).

Yes I use where-am-i as well, but for some reason I didn't notice that the GPS locations were reported as being 1 day old. The initial observation that made me dig into this was that gnome-maps was using the wifi-based locations rather than the gps-based ones.