Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1412)

Unified Diff: chrome/browser/autofill/autofill_manager.cc

Issue 12378055: Make autofill stop depending on InfoBarService (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/autofill/autofill_manager.cc
diff --git a/chrome/browser/autofill/autofill_manager.cc b/chrome/browser/autofill/autofill_manager.cc
index eedeb9c4669b887955b00f59f90f27e90a04f9fe..49df2874ffdd340f256d6c5e9b402f7a2cff75b4 100644
--- a/chrome/browser/autofill/autofill_manager.cc
+++ b/chrome/browser/autofill/autofill_manager.cc
@@ -26,6 +26,7 @@
#include "chrome/browser/autofill/autocheckout_manager.h"
#include "chrome/browser/autofill/autocomplete_history_manager.h"
#include "chrome/browser/autofill/autofill_cc_infobar_delegate.h"
+#include "chrome/browser/autofill/autofill_cc_import_confirmation_handler.h"
#include "chrome/browser/autofill/autofill_country.h"
#include "chrome/browser/autofill/autofill_external_delegate.h"
#include "chrome/browser/autofill/autofill_field.h"
@@ -65,6 +66,9 @@
#include "ui/gfx/rect.h"
typedef PersonalDataManager::GUIDPair GUIDPair;
+
+using autofill::AutofillCCImportConfirmationDelegate;
+using autofill::AutofillCCImportConfirmationHandler;
using base::TimeTicks;
using content::BrowserThread;
using content::RenderViewHost;
@@ -929,8 +933,13 @@ void AutofillManager::ImportFormData(const FormStructure& submitted_form) {
// it.
scoped_ptr<const CreditCard> scoped_credit_card(imported_credit_card);
if (imported_credit_card && web_contents()) {
- AutofillCCInfoBarDelegate::Create(manager_delegate_->GetInfoBarService(),
- scoped_credit_card.release(), personal_data_, metric_logger_.get());
+ scoped_ptr<AutofillCCImportConfirmationDelegate> confirmation_delegate(
+ new AutofillCCImportConfirmationHandler(scoped_credit_card.Pass(),
+ personal_data_,
+ metric_logger_.get()));
+
+ manager_delegate_->ShowAutofillCCImportConfirmation(
+ confirmation_delegate.Pass());
}
}

Powered by Google App Engine
This is Rietveld 408576698