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

chrome/browser/geolocation/core_location_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.
 
 #include "chrome/browser/geolocation/core_location_provider_mac.h"
 #include "chrome/browser/geolocation/core_location_data_provider_mac.h"
 
 #include "base/logging.h"
 #include "base/command_line.h"
 #include "chrome/common/chrome_switches.h"
 
-CoreLocationProviderMac::CoreLocationProviderMac() {
+CoreLocationProviderMac::CoreLocationProviderMac() 
+    : is_updating_(false) {
   data_provider_ = new CoreLocationDataProviderMac();
   data_provider_->AddRef();
 }
 
 CoreLocationProviderMac::~CoreLocationProviderMac() {
   data_provider_->StopUpdating();
   data_provider_->Release();
 }
 
 bool CoreLocationProviderMac::StartProvider(bool high_accuracy) {
-  data_provider_->StartUpdating(this);
+  // StartProvider maybe called multiple times (e.g. to update the high_accuracy hint)
+  // TODO(jknotten): Support high_accuracy hint in underlying data provider.
+  if (is_updating_) 
+    return true;
+  
+  is_updating_ = data_provider_->StartUpdating(this);
   return true;
 }
 
 void CoreLocationProviderMac::StopProvider() {
   data_provider_->StopUpdating();
+  is_updating_ = false;
 }
 
 void CoreLocationProviderMac::GetPosition(Geoposition* position) {
   DCHECK(position);
   *position = position_;
   DCHECK(position->IsInitialized());
 }
 
 void CoreLocationProviderMac::SetPosition(Geoposition* position) {
   DCHECK(position);
   position_ = *position;
   DCHECK(position->IsInitialized());
 
   UpdateListeners();
 }
 
 LocationProviderBase* NewSystemLocationProvider() {
-  if(CommandLine::ForCurrentProcess()
+  if (CommandLine::ForCurrentProcess()

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

I think a more natural break would be:
if (CommandLine::ForCurrentProcess()->HasSwitch(
    switches::kExperimentalLocationFeatures)) {
}

      ->HasSwitch(switches::kExperimentalLocationFeatures)) {
     return new CoreLocationProviderMac;
   }
   return NULL;
 }