New issue
Advanced search Search tips

Issue 717053 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug

Blocked on:
issue 725057

Blocking:
issue 709301


Participants' hotlists:
Geolocation


Sign in to add a comment

Migrate GeolocationDelegate to mojo

Project Member Reported by mcasas@chromium.org, May 1 2017

Issue description

GeolocationDelegate is a pseudo interface overriden by Geolocation 
embedders [1] to provide specific functionality; it is injected via a
static SetGeolocationDelegate() method [2] which holds on to it as a
leaky-singleton.

This situation is preventing the geolocation code to be fully enclosed 
in a mojo boundary and is thus blocking  https://crbug.com/709301  (the
Geolocation servicification bug). 



[1] https://cs.chromium.org/search/?q=device::GeolocationDelegate&type=cs
[2] https://cs.chromium.org/search/?q=GeolocationProvider::SetGeolocationDelegate&type=cs
 
Blockedon: 725057
Labels: DeviceService
Cc: amoylan@chromium.org
Components: Blink>Geolocation
Components: -Blink>Location
Status: WontFix (was: Available)
Closing this bug as it's obsolete now; we're actually just going to eliminate GeolocationDelegate entirely instead.
Components: Internals>Services>Device

Sign in to add a comment