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

Unified Diff: chrome/browser/ui/autofill/autofill_dialog_controller_impl.h

Issue 14129005: Remove "Use billing for shipping" checkbox in favor of item in suggestions menu. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 7 years, 8 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/ui/autofill/autofill_dialog_controller_impl.h
diff --git a/chrome/browser/ui/autofill/autofill_dialog_controller_impl.h b/chrome/browser/ui/autofill/autofill_dialog_controller_impl.h
index f99ee66e7c1f7f9a982d9e252eb4ed565aa589d5..9a6e10062f83e42789e0ef88faa3a06735b13eec 100644
--- a/chrome/browser/ui/autofill/autofill_dialog_controller_impl.h
+++ b/chrome/browser/ui/autofill/autofill_dialog_controller_impl.h
@@ -96,7 +96,6 @@ class AutofillDialogControllerImpl : public AutofillDialogController,
virtual string16 AccountChooserText() const OVERRIDE;
virtual string16 SignInLinkText() const OVERRIDE;
virtual string16 EditSuggestionText() const OVERRIDE;
- virtual string16 UseBillingForShippingText() const OVERRIDE;
virtual string16 CancelButtonText() const OVERRIDE;
virtual string16 ConfirmButtonText() const OVERRIDE;
virtual string16 SaveLocallyText() const OVERRIDE;
@@ -166,8 +165,8 @@ class AutofillDialogControllerImpl : public AutofillDialogController,
const content::NotificationDetails& details) OVERRIDE;
// SuggestionsMenuModelDelegate implementation.
- virtual void SuggestionItemSelected(const SuggestionsMenuModel& model)
- OVERRIDE;
+ virtual void SuggestionItemSelected(SuggestionsMenuModel* model,
+ size_t index) OVERRIDE;
// wallet::WalletClientDelegate implementation.
virtual const AutofillMetrics& GetMetricLogger() const OVERRIDE;
@@ -326,6 +325,8 @@ class AutofillDialogControllerImpl : public AutofillDialogController,
// part of a section is suggested but part must be manually input (e.g. during
// a CVC challenge or when using Autofill's CC section [never stores CVC]).
string16 SuggestionTextForSection(DialogSection section);
+ gfx::Font::FontStyle SuggestionTextStyleForSection(DialogSection section)
+ const;
string16 RequiredActionTextForSection(DialogSection section) const;
gfx::Image SuggestionIconForSection(DialogSection section);
string16 ExtraSuggestionTextForSection(DialogSection section) const;
@@ -372,6 +373,10 @@ class AutofillDialogControllerImpl : public AutofillDialogController,
// the dialog have valid contents.
bool SectionIsValid(DialogSection section) const;
+ // Returns true if |key| refers to a suggestion, as opposed to some control
+ // menu item.
+ bool IsASuggestionItemKey(const std::string& key);
+
// Whether the billing section should be used to fill in the shipping details.
bool ShouldUseBillingForShipping();
« no previous file with comments | « chrome/browser/ui/autofill/autofill_dialog_controller.h ('k') | chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698