Index: chrome/browser/ui/views/extensions/extension_uninstall_dialog_view.cc |
diff --git a/chrome/browser/ui/views/extensions/extension_uninstall_dialog_view.cc b/chrome/browser/ui/views/extensions/extension_uninstall_dialog_view.cc |
index a58b6673344579a47feb6e9345ecd9063244fe51..a50f06092b410272a645e6c707a9b6c884046975 100644 |
--- a/chrome/browser/ui/views/extensions/extension_uninstall_dialog_view.cc |
+++ b/chrome/browser/ui/views/extensions/extension_uninstall_dialog_view.cc |
@@ -84,7 +84,9 @@ class ExtensionUninstallDialogDelegateView : public views::DialogDelegateView { |
virtual base::string16 GetDialogButtonLabel( |
ui::DialogButton button) const OVERRIDE; |
virtual int GetDefaultDialogButton() const OVERRIDE { |
- return ui::DIALOG_BUTTON_CANCEL; |
+ // Default to accept when triggered via chrome://extensions page. |
+ return triggering_extension_ ? |
msw - DO NOT USE
2014/04/19 00:52:58
nit: seems like you could just keep a bool |trigge
Isaac (away)
2014/04/19 01:06:19
Switched to bool.
|
+ ui::DIALOG_BUTTON_CANCEL : ui::DIALOG_BUTTON_OK; |
} |
virtual bool Accept() OVERRIDE; |
virtual bool Cancel() OVERRIDE; |
@@ -104,6 +106,7 @@ class ExtensionUninstallDialogDelegateView : public views::DialogDelegateView { |
views::ImageView* icon_; |
views::Label* heading_; |
+ const extensions::Extension* triggering_extension_; |
DISALLOW_COPY_AND_ASSIGN(ExtensionUninstallDialogDelegateView); |
}; |
@@ -172,6 +175,8 @@ ExtensionUninstallDialogDelegateView::ExtensionUninstallDialogDelegateView( |
heading_->SetMultiLine(true); |
heading_->SetHorizontalAlignment(gfx::ALIGN_LEFT); |
AddChildView(heading_); |
+ |
+ triggering_extension_ = triggering_extension; |
} |
ExtensionUninstallDialogDelegateView::~ExtensionUninstallDialogDelegateView() { |