Dispatch geolocation IPCs on the UI thread. Aside from simplifying the code to avoid a lot of threa…

chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc

 // Copyright 2013 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/ui/autofill/autofill_dialog_controller_impl.h"
 
 #include <algorithm>
 #include <map>
 #include <string>
 
@@ skipping 287 matching lines @@
     case wallet::WalletItems::MaskedInstrument::VALID:
     case wallet::WalletItems::MaskedInstrument::PENDING:
     case wallet::WalletItems::MaskedInstrument::EXPIRED:
     case wallet::WalletItems::MaskedInstrument::BILLING_INCOMPLETE:
       return true;
     default:
       return false;
   }
 }
 
-// Signals that the user has opted in to geolocation services.  Factored out
-// into a separate method because all interaction with the geolocation provider
-// needs to happen on the IO thread, which is not the thread
-// AutofillDialogViewDelegate lives on.
-void UserDidOptIntoLocationServices() {
-  content::GeolocationProvider::GetInstance()->UserDidOptIntoLocationServices();
-}
-
 // Loops through |addresses_| comparing to |address| ignoring ID. If a match
 // is not found, NULL is returned.
 const wallet::Address* FindDuplicateAddress(
     const std::vector<wallet::Address*>& addresses,
     const wallet::Address& address) {
   for (size_t i = 0; i < addresses.size(); ++i) {
     if (addresses[i]->EqualsIgnoreID(address))
       return addresses[i];
   }
   return NULL;
@@ skipping 3292 matching lines @@
     view_->UpdateNotificationArea();
   }
 }
 
 bool AutofillDialogControllerImpl::AreLegalDocumentsCurrent() const {
   return has_accepted_legal_documents_ ||
       (wallet_items_ && wallet_items_->legal_documents().empty());
 }
 
 void AutofillDialogControllerImpl::AcceptLegalTerms() {
-  content::BrowserThread::PostTask(
-      content::BrowserThread::IO, FROM_HERE,
-      base::Bind(&UserDidOptIntoLocationServices));
+  content::GeolocationProvider::GetInstance()->UserDidOptIntoLocationServices();
   PrefService* local_state = g_browser_process->local_state();
   ListPrefUpdate accepted(
       local_state, ::prefs::kAutofillDialogWalletLocationAcceptance);
   accepted->AppendIfNotPresent(new base::StringValue(
       account_chooser_model_->GetActiveWalletAccountName()));
 
   if (AreLegalDocumentsCurrent()) {
     LoadRiskFingerprintData();
   } else {
     GetWalletClient()->AcceptLegalDocuments(
@@ skipping 440 matching lines @@
   view_->UpdateButtonStrip();
 }
 
 void AutofillDialogControllerImpl::FetchWalletCookie() {
   net::URLRequestContextGetter* request_context = profile_->GetRequestContext();
   signin_helper_.reset(new wallet::WalletSigninHelper(this, request_context));
   signin_helper_->StartWalletCookieValueFetch();
 }
 
 }  // namespace autofill