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

Side by Side Diff: chrome/browser/views/autofill_profiles_view_win.cc

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
« no previous file with comments | « chrome/browser/gtk/options/content_page_gtk.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 #include "chrome/browser/views/autofill_profiles_view_win.h" 4 #include "chrome/browser/views/autofill_profiles_view_win.h"
5 5
6 #include <vsstyle.h> 6 #include <vsstyle.h>
7 #include <vssym32.h> 7 #include <vssym32.h>
8 8
9 #include "app/l10n_util.h" 9 #include "app/l10n_util.h"
10 #include "app/resource_bundle.h" 10 #include "app/resource_bundle.h"
(...skipping 1417 matching lines...) Expand 10 before | Expand all | Expand 10 after
1428 1428
1429 gfx::Size border = gfx::NativeTheme::instance()->GetThemeBorderSize( 1429 gfx::Size border = gfx::NativeTheme::instance()->GetThemeBorderSize(
1430 gfx::NativeTheme::LIST); 1430 gfx::NativeTheme::LIST);
1431 border.set_width(border.width() + kPanelHorizMargin); 1431 border.set_width(border.width() + kPanelHorizMargin);
1432 lb.Inset(border.width(), border.height()); 1432 lb.Inset(border.width(), border.height());
1433 scroll_view_->SetBounds(lb); 1433 scroll_view_->SetBounds(lb);
1434 scroll_view_->Layout(); 1434 scroll_view_->Layout();
1435 } 1435 }
1436 1436
1437 // Declared in "chrome/browser/autofill/autofill_dialog.h" 1437 // Declared in "chrome/browser/autofill/autofill_dialog.h"
1438 void ShowAutoFillDialog(gfx::NativeWindow parent, 1438 // TODO(georgey): Need to update implementation to match new interface for
1439 // |imported_profile| and |imported_credit_card| parameters.
1440 // See http://crbug.com/41010
1441 void ShowAutoFillDialog(gfx::NativeView parent,
1439 AutoFillDialogObserver* observer, 1442 AutoFillDialogObserver* observer,
1440 Profile* profile) { 1443 Profile* profile,
1444 AutoFillProfile* imported_profile,
1445 CreditCard* imported_credit_card) {
1441 DCHECK(profile); 1446 DCHECK(profile);
1442 1447
1443 // It's possible we haven't shown the InfoBar yet, but if the user is in the 1448 // It's possible we haven't shown the InfoBar yet, but if the user is in the
1444 // AutoFill dialog, she doesn't need to be asked to enable or disable 1449 // AutoFill dialog, she doesn't need to be asked to enable or disable
1445 // AutoFill. 1450 // AutoFill.
1446 profile->GetPrefs()->SetBoolean(prefs::kAutoFillInfoBarShown, true); 1451 profile->GetPrefs()->SetBoolean(prefs::kAutoFillInfoBarShown, true);
1447 1452
1448 PersonalDataManager* personal_data_manager = 1453 PersonalDataManager* personal_data_manager =
1449 profile->GetPersonalDataManager(); 1454 profile->GetPersonalDataManager();
1450 DCHECK(personal_data_manager); 1455 DCHECK(personal_data_manager);
1451 AutoFillProfilesView::Show(parent, observer, personal_data_manager, 1456 AutoFillProfilesView::Show(parent, observer, personal_data_manager,
1452 profile->GetPrefs()); 1457 profile->GetPrefs());
1453 } 1458 }
OLDNEW
« no previous file with comments | « chrome/browser/gtk/options/content_page_gtk.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698