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

Unified Diff: chrome/browser/ui/views/apps/app_info_dialog/app_info_permissions_panel.cc

Issue 2744463002: Add VectorIconButton functionality to ImageButton. (Closed)
Patch Set: fix cros build Created 3 years, 9 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/apps/app_info_dialog/app_info_permissions_panel.cc
diff --git a/chrome/browser/ui/views/apps/app_info_dialog/app_info_permissions_panel.cc b/chrome/browser/ui/views/apps/app_info_dialog/app_info_permissions_panel.cc
index 6dbc4988a61e3b8117f67afd476f079c12ac9219..d0d111b7649535f95bcea03665e3d77e2abda871 100644
--- a/chrome/browser/ui/views/apps/app_info_dialog/app_info_permissions_panel.cc
+++ b/chrome/browser/ui/views/apps/app_info_dialog/app_info_permissions_panel.cc
@@ -45,7 +45,8 @@ const int kIndentationBeforeNestedBullet = 13;
// Creates a close button that calls |callback| on click and can be placed to
// the right of a bullet in the permissions list. The alt-text is set to a
// revoke message containing the given |permission_message|.
-class RevokeButton : public views::ImageButton, public views::ButtonListener {
+class RevokeButton : public views::ImageButton,
+ public views::ImageButtonDelegate {
public:
explicit RevokeButton(const base::Closure& callback,
base::string16 permission_message)
@@ -70,7 +71,7 @@ class RevokeButton : public views::ImageButton, public views::ButtonListener {
~RevokeButton() override {}
private:
- // Overridden from views::ButtonListener.
+ // views::ImageButtonDelegate:
void ButtonPressed(views::Button* sender, const ui::Event& event) override {
DCHECK_EQ(this, sender);
if (!callback_.is_null())

Powered by Google App Engine
This is Rietveld 408576698