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

Unified Diff: components/autofill/core/browser/autofill_external_delegate.h

Issue 2531223003: Expanded Autofill Credit Card Popup Layout Experiment in Android. (Closed)
Patch Set: Fixes compile error. Created 4 years 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: components/autofill/core/browser/autofill_external_delegate.h
diff --git a/components/autofill/core/browser/autofill_external_delegate.h b/components/autofill/core/browser/autofill_external_delegate.h
index 4afbb92de4937d9f53a42937dec1d34a0e72f945..6a9807a05054adb418eac0c65fb5e70e15e84e63 100644
--- a/components/autofill/core/browser/autofill_external_delegate.h
+++ b/components/autofill/core/browser/autofill_external_delegate.h
@@ -51,6 +51,9 @@ class AutofillExternalDelegate : public AutofillPopupDelegate {
base::string16* body) override;
bool RemoveSuggestion(const base::string16& value, int identifier) override;
void ClearPreviewedForm() override;
+ // Returns false for all popups prior to |onQuery|, true for credit card
+ // popups after call to |onQuery|.
+ bool IsCreditCardPopup() override;
// Records and associates a query_id with web form data. Called
// when the renderer posts an Autofill query to the browser. |bounds|
@@ -115,6 +118,9 @@ class AutofillExternalDelegate : public AutofillPopupDelegate {
// version.
void InsertDataListValues(std::vector<Suggestion>* suggestions);
+ // Returns the text (i.e. |Suggestion| value) for Chrome autofill options.
+ base::string16 GetSettingsSuggestionValue() const;
+
AutofillManager* manager_; // weak.
// Provides driver-level context to the shared code of the component. Must
@@ -139,8 +145,8 @@ class AutofillExternalDelegate : public AutofillPopupDelegate {
// currently editing? Used to keep track of state for metrics logging.
bool has_shown_popup_for_current_edit_;
- // FIXME
bool should_show_scan_credit_card_;
+ bool is_credit_card_popup_;
// Whether the credit card signin promo should be shown to the user.
bool should_show_cc_signin_promo_;

Powered by Google App Engine
This is Rietveld 408576698