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

chrome/browser/geolocation/core_location_provider_mac.mm

Index: chrome/browser/geolocation/core_location_provider_mac.mm
diff --git a/chrome/browser/geolocation/core_location_provider_mac.mm b/chrome/browser/geolocation/core_location_provider_mac.mm
index deb92c90d7130c6b50b8170e03123939b06b6823..dbcd7d07926a7e9c0662faf3c4919abccb7fa031 100644
--- a/chrome/browser/geolocation/core_location_provider_mac.mm
+++ b/chrome/browser/geolocation/core_location_provider_mac.mm
@@ -9,7 +9,8 @@
 #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();
 }
@@ -20,12 +21,19 @@ CoreLocationProviderMac::~CoreLocationProviderMac() {
 }
 
 bool CoreLocationProviderMac::StartProvider(bool high_accuracy) {
-  data_provider_->StartUpdating(this);
+  // StartProvider maybe called multiple times. For example, 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) {
@@ -43,8 +51,8 @@ void CoreLocationProviderMac::SetPosition(Geoposition* position) {
 }
 
 LocationProviderBase* NewSystemLocationProvider() {
-  if(CommandLine::ForCurrentProcess()
-     ->HasSwitch(switches::kExperimentalLocationFeatures)) {
+  if (CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kExperimentalLocationFeatures)) {
     return new CoreLocationProviderMac;
   }
   return NULL;