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

Unified Diff: chrome/browser/extensions/extension_install_prompt.h

Issue 11087071: Making ShowExtensionInstallDialog a callback (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 2 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/extensions/extension_install_prompt.h
diff --git a/chrome/browser/extensions/extension_install_prompt.h b/chrome/browser/extensions/extension_install_prompt.h
index 480dfd062caac34e19bbfd38b701a695ab08afc6..d4e728bf1154df492bd85b8743f0c70903d941c0 100644
--- a/chrome/browser/extensions/extension_install_prompt.h
+++ b/chrome/browser/extensions/extension_install_prompt.h
@@ -21,6 +21,7 @@
#include "ui/gfx/native_widget_types.h"
class Browser;
+class ExtensionInstallDialog;
class ExtensionInstallUI;
class InfoBarDelegate;
class MessageLoop;
@@ -199,16 +200,19 @@ class ExtensionInstallPrompt : public ImageLoadingTracker::Observer,
// the webstore should proceed.
//
// We *MUST* eventually call either Proceed() or Abort() on |delegate|.
- virtual void ConfirmWebstoreInstall(Delegate* delegate,
- const extensions::Extension* extension,
- const SkBitmap* icon);
+ virtual void ConfirmWebstoreInstall(
+ Delegate* delegate,
+ const extensions::Extension* extension,
+ const SkBitmap* icon,
+ scoped_refptr<ExtensionInstallDialog> dialog);
// This is called by the installer to verify whether the installation should
// proceed. This is declared virtual for testing.
//
// We *MUST* eventually call either Proceed() or Abort() on |delegate|.
virtual void ConfirmInstall(Delegate* delegate,
- const extensions::Extension* extension);
+ const extensions::Extension* extension,
+ scoped_refptr<ExtensionInstallDialog> dialog);
// This is called by the app handler launcher to verify whether the app
// should be re-enabled. This is declared virtual for testing.
@@ -311,6 +315,9 @@ class ExtensionInstallPrompt : public ImageLoadingTracker::Observer,
// Keeps track of extension images being loaded on the File thread for the
// purpose of showing the install UI.
ImageLoadingTracker tracker_;
+
+ // Used to show the confirm dialog.
+ scoped_refptr<ExtensionInstallDialog> dialog_;
};
namespace chrome {

Powered by Google App Engine
This is Rietveld 408576698