Fix incorrect assertion following the move of Geolocation provider in to its own thread.

chrome/browser/geolocation/core_location_data_provider_mac.mm

 // Copyright (c) 2010 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 contains the class definitions for the CoreLocation data provider
 // class and the accompanying Objective C wrapper class. This data provider
 // is used to allow the CoreLocation wrapper to run on the UI thread, since
 // CLLocationManager's start and stop updating methods must be called from a
 // thread with an active NSRunLoop.  Currently only the UI thread appears to
 // fill that requirement.
 
 #include "chrome/browser/geolocation/core_location_data_provider_mac.h"
 #include "chrome/browser/geolocation/core_location_provider_mac.h"
+#include "chrome/browser/geolocation/geolocation_provider.h"
 #include "base/logging.h"
 #include "base/time.h"
 
 // A few required declarations since the CoreLocation headers are not available
 // with the Mac OS X 10.5 SDK.
 // TODO(jorgevillatoro): Remove these declarations when we build against 10.6
 
 // This idea was borrowed from wifi_data_provider_corewlan_mac.mm
 typedef double CLLocationDegrees;
 typedef double CLLocationAccuracy;
@@ skipping 152 matching lines @@
 
     locationManagerClass_ = [bundle_ classNamed:@"CLLocationManager"];
   }
 
   return YES;
 }
 
 @end
 
 CoreLocationDataProviderMac::CoreLocationDataProviderMac() {
-  if(!BrowserThread::GetCurrentThreadIdentifier(&origin_thread_id_))
+  if(MessageLoop::current() != GeolocationProvider::GetInstance()->message_loop())

[bulach, 2011/01/06 12:44:59]

nit:space after if, and >80cols

if (MessageLoop::current() !=
    GeolocationProvider::GetInstance()->message_loop()) {
}

     NOTREACHED() <<
-         "CoreLocation data provider must be created in a valid BrowserThread.";
+         "CoreLocation data provider must be created on the Geolocation thread.";

[bulach, 2011/01/06 12:44:59]

nit: >80cols.

   provider_ = NULL;
   wrapper_.reset([[CoreLocationWrapperMac alloc] initWithDataProvider:this]);
 }
 
 CoreLocationDataProviderMac::~CoreLocationDataProviderMac() {
 }
 
 // Returns true if the CoreLocation wrapper can load the framework and
-// location services are enabled.  The pointer argument will only be accessed
+// location services are enabled.  The pointer toargument will only be accessed

[bulach, 2011/01/06 12:44:59]

s/to//

 // in the origin thread.
 bool CoreLocationDataProviderMac::
     StartUpdating(CoreLocationProviderMac* provider) {
   DCHECK(provider);
   DCHECK(!provider_) << "StartUpdating called twice";
   if(![wrapper_ locationDataAvailable]) return false;
   provider_ = provider;
   BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
      NewRunnableMethod(this, &CoreLocationDataProviderMac::StartUpdatingTask));
   return true;
 }
 
 // Clears provider_ so that any leftover messages from CoreLocation get ignored
 void CoreLocationDataProviderMac::StopUpdating() {
   provider_ = NULL;
   BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
               NewRunnableMethod(this,
                                &CoreLocationDataProviderMac::StopUpdatingTask));
 }
 
 void CoreLocationDataProviderMac::UpdatePosition(Geoposition *position) {
-  BrowserThread::PostTask(origin_thread_id_, FROM_HERE,
+  GeolocationProvider::GetInstance()->message_loop()->PostTask(FROM_HERE,
               NewRunnableMethod(this,
                                &CoreLocationDataProviderMac::PositionUpdated,
                                *position));
 }
 
 // Runs in BrowserThread::UI
 void CoreLocationDataProviderMac::StartUpdatingTask() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   [wrapper_ startLocation];
 }
 
 // Runs in BrowserThread::UI
 void CoreLocationDataProviderMac::StopUpdatingTask() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   [wrapper_ stopLocation];
 }
 
 void CoreLocationDataProviderMac::PositionUpdated(Geoposition position) {
-  DCHECK(BrowserThread::CurrentlyOn(origin_thread_id_));
+  DCHECK(MessageLoop::current() == GeolocationProvider::GetInstance()->message_loop());

[bulach, 2011/01/06 12:44:59]

nit: >80cols

   if(provider_)
     provider_->SetPosition(&position);
 }