Delete todo to add threading in GpsLocationProviderLinux.

content/browser/geolocation/gps_location_provider_linux.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This file declares GPS providers that run on linux. Currently, just uses
 // the libgps (gpsd) API. Public for testing only - for normal usage this
 // header should not be required, as location_provider.h declares the needed
 // factory function.
 
 #ifndef CONTENT_BROWSER_GEOLOCATION_GPS_LOCATION_PROVIDER_LINUX_H_
@@ skipping 38 matching lines @@
 
   scoped_ptr<gps_data_t> gps_data_;
   bool is_open_;
 #endif  // defined(USE_LIBGPS)
 
  private:
   DISALLOW_COPY_AND_ASSIGN(LibGps);
 };
 
 // Location provider for Linux, that uses libgps/gpsd to obtain position fixes.
-// TODO(joth): Currently this runs entirely in the client thread (i.e. Chrome's
-// IO thread). As the older libgps API is not designed to support polling,
-// there's a chance it could block, so better move this into its own worker
-// thread.
 class CONTENT_EXPORT GpsLocationProviderLinux : public LocationProviderBase {
  public:
   typedef LibGps* (*LibGpsFactory)();
   // |factory| will be used to create the gpsd client library wrapper. (Note
   // NewSystemLocationProvider() will use the default factory).
   explicit GpsLocationProviderLinux(LibGpsFactory libgps_factory);
   virtual ~GpsLocationProviderLinux();
 
   void SetGpsdReconnectIntervalMillis(int value) {
     gpsd_reconnect_interval_millis_ = value;
@@ skipping 27 matching lines @@
   scoped_ptr<LibGps> gps_;
   Geoposition position_;
 
   // Holder for the tasks which run on the thread; takes care of cleanup.
   base::WeakPtrFactory<GpsLocationProviderLinux> weak_factory_;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_GEOLOCATION_GPS_LOCATION_PROVIDER_LINUX_H_