From 58db2f88917448f794deeee45868137d9098339a Mon Sep 17 00:00:00 2001 From: "estade@chromium.org" Date: Wed, 14 Oct 2009 20:12:50 +0000 Subject: Make the multiple download request dialog an infobar. The icon is a placeholder until Glen makes a pretty one. BUG=24047 TEST=go to skypher.com/SkyLined/Repro/Chrome/carpet bombing/repro.html allow, deny, closing infobar, and closing tab all work as expected Review URL: http://codereview.chromium.org/275011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29006 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/extensions/extension_disabled_infobar_delegate.cc | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'chrome/browser/extensions') diff --git a/chrome/browser/extensions/extension_disabled_infobar_delegate.cc b/chrome/browser/extensions/extension_disabled_infobar_delegate.cc index 14cc3e7..8e4de1a 100644 --- a/chrome/browser/extensions/extension_disabled_infobar_delegate.cc +++ b/chrome/browser/extensions/extension_disabled_infobar_delegate.cc @@ -37,6 +37,9 @@ class ExtensionDisabledDialogDelegate NewRunnableMethod(this, &ExtensionDisabledDialogDelegate::Start)); } + virtual ~ExtensionDisabledDialogDelegate() { + } + // ExtensionInstallUI::Delegate virtual void ContinueInstall() { service_->EnableExtension(extension_->id()); @@ -89,8 +92,7 @@ class ExtensionDisabledInfobarDelegate registrar_.Add(this, NotificationType::EXTENSION_UNLOADED_DISABLED, Source(service)); } - virtual void InfoBarClosed() { - delete this; + virtual ~ExtensionDisabledInfobarDelegate() { } virtual std::wstring GetMessageText() const { return l10n_util::GetStringF(IDS_EXTENSION_DISABLED_INFOBAR_LABEL, -- cgit v1.1