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

Unified Diff: chrome/browser/ui/views/autofill/autofill_dialog_views.h

Issue 13625002: Change the behavior of the [X] Save details to Wallet checkbox notification to: (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/views/autofill/autofill_dialog_views.h
diff --git a/chrome/browser/ui/views/autofill/autofill_dialog_views.h b/chrome/browser/ui/views/autofill/autofill_dialog_views.h
index 1f5eacc9d409a8e82a3c0ed38dde2e9e04834ff7..331ddc9820a37314326a6dbbcfe568c8768b8b29 100644
--- a/chrome/browser/ui/views/autofill/autofill_dialog_views.h
+++ b/chrome/browser/ui/views/autofill/autofill_dialog_views.h
@@ -79,7 +79,6 @@ class AutofillDialogViews : public AutofillDialogView,
DetailOutputMap* output) OVERRIDE;
virtual string16 GetCvc() OVERRIDE;
virtual bool UseBillingForShipping() OVERRIDE;
- virtual bool SaveDetailsInWallet() OVERRIDE;
virtual bool SaveDetailsLocally() OVERRIDE;
virtual const content::NavigationController* ShowSignIn() OVERRIDE;
virtual void HideSignIn() OVERRIDE;
@@ -208,33 +207,37 @@ class AutofillDialogViews : public AutofillDialogView,
};
// An area for notifications. Some notifications point at the account chooser.
- class NotificationArea : public views::View {
+ class NotificationArea : public views::View,
+ public views::ButtonListener {
public:
- NotificationArea();
+ explicit NotificationArea(AutofillDialogController* controller);
virtual ~NotificationArea();
- void set_arrow_centering_anchor(
- const base::WeakPtr<views::View>& arrow_centering_anchor) {
- arrow_centering_anchor_ = arrow_centering_anchor;
- }
-
// Displays the given notifications.
void SetNotifications(const std::vector<DialogNotification>& notifications);
- // Returns true if the checkbox exists and is checked. Currently, the
- // notification area only supports showing a checkbox on the topmost
- // notification.
- bool CheckboxIsChecked() const;
-
// views::View implementation.
virtual std::string GetClassName() const OVERRIDE;
virtual void OnPaint(gfx::Canvas* canvas) OVERRIDE;
+ // views::ButtonListener implementation:
+ virtual void ButtonPressed(views::Button* sender,
+ const ui::Event& event) OVERRIDE;
+
+ void set_arrow_centering_anchor(
+ const base::WeakPtr<views::View>& arrow_centering_anchor) {
+ arrow_centering_anchor_ = arrow_centering_anchor;
+ }
+
private:
// Utility function for determining whether an arrow should be drawn
// pointing at |arrow_centering_anchor_|.
bool HasArrow();
+ // A reference to the controller than owns this view. Used to report when
+ // checkboxes change their values.
+ AutofillDialogController* controller_; // weak
+
// The currently showing checkbox, or NULL if none exists.
views::Checkbox* checkbox_; // weak
« no previous file with comments | « chrome/browser/ui/autofill/autofill_dialog_view.h ('k') | chrome/browser/ui/views/autofill/autofill_dialog_views.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698