[Webkit-unassigned] [Bug 195996] New: Geolocation request not complete when watch request was started in a different web process

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=195996

            Bug ID: 195996
           Summary: Geolocation request not complete when watch request
                    was started in a different web process
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebKit2
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: cgarcia at igalia.com
                CC: achristensen at apple.com, sam at webkit.org

This can be easily reproduced using the web inspector to start the geolocation requests:

1.- Open a browser window and load any https page.
2.- Open the inspector and make a watch request from the console
  navigator.geolocation.watchPosition(function(p) { alert(p); });
3.- Allow the request and close the alert.
4.- Open a new browser window (not a tab, since we need both web view to be visible) and load a different https page
5.- Open the inspector and make a location request from the console
  navigator.geolocation.getCurrentPosition(function(p) { alert(p) })
6.- Allow the request

The alert is not shown in this case. This is because in WebGeolocationManagerProxy::startUpdating() we do nothing when the provider is already updating. We should reply with a DidChangePosition using the last known position, if available. If we are updating, but we still don't have a known position, the request will be completed when WebGeolocationManagerProxy::providerDidChangePosition() is called since it always notifies all web processes.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20190320/7c542c83/attachment.html>