summaryrefslogtreecommitdiffstats
path: root/content/browser/geolocation/location_arbitrator_impl.cc
Commit message (Collapse)AuthorAgeFilesLines
* Convert string16 to base::string16 in content.brettw@chromium.org2013-12-031-1/+1
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/102593002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238447 0039d316-1c4b-4281-b951-d872f2087c98
* Geolocation: delete GpsLocationProvider and third_party/gpsd/mvanouwerkerk@chromium.org2013-10-041-1/+1
| | | | | | | | | | | | | | | The GpsLocationProvider is not actually used anywhere, and has most likely bit rotted after years of no maintenance. It is brittle code written for a specific version (3.1) of gpsd that is now outdated. The only Chrome version that had this feature enabled (ChromeOS) makes no use of it as no hardware was ever shipped that used GPS. TBR=jam,brettw Review URL: https://codereview.chromium.org/25752005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@227104 0039d316-1c4b-4281-b951-d872f2087c98
* Rename GeolocationArbitrator to LocationArbitrator.mvanouwerkerk@chromium.org2013-09-101-20/+20
| | | | | | | | It was generally referred to in file names as location_arbitrator, but the class was called GeolocationArbitrator. Review URL: https://chromiumcodereview.appspot.com/23844003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@222338 0039d316-1c4b-4281-b951-d872f2087c98
* Delete Mac Core Geolocation provider.mvanouwerkerk@chromium.org2013-08-141-1/+1
| | | | | | | | | | This experimental feature is hidden behind the experimental-location-features flag. It was never fully launched and has very little usage. Please refer to the bug for more details. BUG=134075 Review URL: https://chromiumcodereview.appspot.com/22809002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@217544 0039d316-1c4b-4281-b951-d872f2087c98
* Delete the Windows 7 Geolocation provider.mvanouwerkerk@chromium.org2013-08-121-0/+4
| | | | | | | | | | This experimental feature is hidden behind the experimental-location-features flag and was never fully developed and has very little usage. Please refer to the bug for more details. BUG=134075 Review URL: https://chromiumcodereview.appspot.com/22674006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@216980 0039d316-1c4b-4281-b951-d872f2087c98
* content: Migrate from googleurl/ includes to url/ ones.tfarina@chromium.org2013-07-091-1/+1
| | | | | | | | | BUG=229660 TBR=darin@chromium.org Review URL: https://codereview.chromium.org/18868005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@210626 0039d316-1c4b-4281-b951-d872f2087c98
* location: Refactor LocationProvider.sadrul@chromium.org2013-06-121-13/+20
| | | | | | | | | | | | | | | | | | | | | | Expose content::LocationProvider to the embedder. This allows the content embedder to provider different system location providers. Notable changes: * Introduce content::LocationProvider in the public content API. Move the virtual functions of LocationProviderBase into this interface. * Simplify LocationProviderBase. Instead of using a listener interface for LocationProvider, provide callbacks to LocationProvider that get triggered when new location data is available. * Introduce ContentBrowserClient::CreateSystemLocationProvider. This is the entry point for the embedder to provide a custom LocationProvider. BUG=247587 R=bulach@chromium.org, jam@chromium.org Review URL: https://codereview.chromium.org/15774013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@205708 0039d316-1c4b-4281-b951-d872f2087c98
* Clear Reference Location State When Providers are Stoppedrobliao@chromium.org2013-06-041-0/+6
| | | | | | | | | | | | | | | | The location arbitrator relies on pointer comparisons to identify which location provider provided a reported location. Now, we clear this state when the providers are shut down. This prevents a bug where a stale location prevented the arbitrator from updating its clients with a newer location. BUG=240956 Review URL: https://chromiumcodereview.appspot.com/15968009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@203924 0039d316-1c4b-4281-b951-d872f2087c98
* Update content/ to use scoped_refptr<T>::get() rather than implicit ↵rsleevi@chromium.org2013-06-021-1/+1
| | | | | | | | | | | | | "operator T*" Linux fixes BUG=110610 TBR=darin Review URL: https://chromiumcodereview.appspot.com/16294003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@203624 0039d316-1c4b-4281-b951-d872f2087c98
* [Geolocation] Fix a race condition in GeolocationArbitratorImplisherman@chromium.org2013-05-161-2/+5
| | | | | | | | | | | | | | | | Previously, the following sequence of events was possible: (1) StartProviders() posts an asynchronous task T to continue startup. (2) StopProviders() is called. (3) T comes back, and startup continues. Now, step (3) will short-circuit so that startup does not continue if the asynchronous continuation comes in after StopProviders() is called. BUG=166596 Review URL: https://chromiumcodereview.appspot.com/14864017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@200399 0039d316-1c4b-4281-b951-d872f2087c98
* Expose GeolocationProvider in Content API. This removes the recently ↵jam@chromium.org2013-05-071-7/+6
| | | | | | | | | | | | | introduced usage of internal content classes from chrome. I simplified things a bit in the process. I removed GeolocationObserverOptions since all we need is a boolean (it's been like that for a long time). I also switched from the one-method GeolocationObserver to a callback. This also means that there's one way to get location info, compared to before when there were two ways (interface+callback). BUG=237552 R=bulach@chromium.org, joth@chromium.org Review URL: https://codereview.chromium.org/14688015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@198751 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: Remove unnecessary ".get()" from scoped_ptrs<>.erg@chromium.org2013-04-171-1/+1
| | | | | | | | | | | In r174057, ajwong@ added support for implicit testing to scoped_ptr<>. Removes these in content/. BUG=232084 Review URL: https://codereview.chromium.org/14081010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@194608 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: use MockGeolocationArbitrator to test GeolocationProviderjknotten@chromium.org2012-11-301-0/+185
This patch introduces a MockGeolocationArbitrator class for use with the GeolocationProvider unittests. This avoids having to instantiate dependencies of the GeolocationArbitrator to test the GeolocationProvider. This will make it easier to add a unit test for https://codereview.chromium.org/11312210/ TEST=Existing BUG= Review URL: https://chromiumcodereview.appspot.com/11412194 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170444 0039d316-1c4b-4281-b951-d872f2087c98