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

Unified Diff: chrome/browser/extensions/extension_browsertest.cc

Issue 11087071: Making ShowExtensionInstallDialog a callback (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address review comments 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_browsertest.cc
diff --git a/chrome/browser/extensions/extension_browsertest.cc b/chrome/browser/extensions/extension_browsertest.cc
index bbb3aa5e61d05a07fcdcc14b016ea82dedc32c23..83efb868d1bff82ad81bc1a0caad25ea40afce1f 100644
--- a/chrome/browser/extensions/extension_browsertest.cc
+++ b/chrome/browser/extensions/extension_browsertest.cc
@@ -258,7 +258,9 @@ class MockAbortExtensionInstallPrompt : public ExtensionInstallPrompt {
}
// Simulate a user abort on an extension installation.
- virtual void ConfirmInstall(Delegate* delegate, const Extension* extension) {
+ virtual void ConfirmInstall(Delegate* delegate,
+ const Extension* extension,
+ scoped_refptr<ExtensionInstallDialog> dialog) {
delegate->InstallUIAbort(true);
MessageLoopForUI::current()->Quit();
}
@@ -276,7 +278,9 @@ class MockAutoConfirmExtensionInstallPrompt : public ExtensionInstallPrompt {
Profile* profile) : ExtensionInstallPrompt(parent, navigator, profile) {}
// Proceed without confirmation prompt.
- virtual void ConfirmInstall(Delegate* delegate, const Extension* extension) {
+ virtual void ConfirmInstall(Delegate* delegate,
+ const Extension* extension,
+ scoped_refptr<ExtensionInstallDialog> dialog) {
delegate->InstallUIProceed();
}
};

Powered by Google App Engine
This is Rietveld 408576698