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

Side by Side Diff: chrome/browser/autofill/autofill_dialog_mac.mm

Issue 1930002: AutoFill profile shouldn't be saved when cancelled during initial setup. (Closed)
Patch Set: Addressing review comments. Polishing unit test comments. Created 10 years, 7 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #import "chrome/browser/autofill/autofill_dialog_controller_mac.h" 5 #import "chrome/browser/autofill/autofill_dialog_controller_mac.h"
6 #include "chrome/browser/autofill/autofill_dialog.h" 6 #include "chrome/browser/autofill/autofill_dialog.h"
7 #include "chrome/browser/pref_service.h" 7 #include "chrome/browser/pref_service.h"
8 #include "chrome/browser/profile.h" 8 #include "chrome/browser/profile.h"
9 #include "chrome/common/pref_names.h" 9 #include "chrome/common/pref_names.h"
10 10
11 // Mac implementation of |ShowAutoFillDialog| interface defined in 11 // Mac implementation of |ShowAutoFillDialog| interface defined in
12 // |chrome/browser/autofill/autofill_dialog.h|. 12 // |chrome/browser/autofill/autofill_dialog.h|.
13 void ShowAutoFillDialog(AutoFillDialogObserver* observer, 13 void ShowAutoFillDialog(gfx::NativeView parent,
14 const std::vector<AutoFillProfile*>& profiles, 14 AutoFillDialogObserver* observer,
15 const std::vector<CreditCard*>& credit_cards, 15 Profile* profile,
16 Profile *profile) { 16 AutoFillProfile* imported_profile,
17 CreditCard* imported_credit_card) {
17 // It's possible we haven't shown the InfoBar yet, but if the user is in the 18 // It's possible we haven't shown the InfoBar yet, but if the user is in the
18 // AutoFill dialog, she doesn't need to be asked to enable or disable 19 // AutoFill dialog, she doesn't need to be asked to enable or disable
19 // AutoFill. 20 // AutoFill.
20 profile->GetPrefs()->SetBoolean(prefs::kAutoFillInfoBarShown, true); 21 profile->GetPrefs()->SetBoolean(prefs::kAutoFillInfoBarShown, true);
21 22
22 [AutoFillDialogController 23 [AutoFillDialogController
23 showAutoFillDialogWithObserver:observer 24 showAutoFillDialogWithObserver:observer
24 autoFillProfiles:profiles 25 profile:profile
25 creditCards:credit_cards 26 importedProfile:imported_profile
26 profile:profile]; 27 importedCreditCard:imported_credit_card];
27 } 28 }
28
OLDNEW
« no previous file with comments | « chrome/browser/autofill/autofill_dialog_gtk.cc ('k') | chrome/browser/autofill/autofill_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698