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

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

Issue 4727001: Split the private webstore install API into two parts.... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 1 month 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_webstore_private_api.h
===================================================================
--- chrome/browser/extensions/extension_webstore_private_api.h (revision 65457)
+++ chrome/browser/extensions/extension_webstore_private_api.h (working copy)
@@ -25,14 +25,23 @@
static void SetTestingBrowserSignin(BrowserSignin* signin);
};
-class InstallFunction : public SyncExtensionFunction {
+class BeginInstallFunction : public SyncExtensionFunction {
public:
+ // For use only in tests - sets a flag that can cause this function to ignore
+ // the normal requirement that it is called during a user gesture.
+ static void SetIgnoreUserGestureForTests(bool ignore);
+ protected:
+ virtual bool RunImpl();
+ DECLARE_EXTENSION_FUNCTION_NAME("webstorePrivate.beginInstall");
+};
+
+class CompleteInstallFunction : public SyncExtensionFunction {
+ public:
+ // This changes the base of the download url to the crx file to be installed.
static void SetTestingInstallBaseUrl(const char* testing_install_base_url);
-
protected:
- ~InstallFunction() {}
virtual bool RunImpl();
- DECLARE_EXTENSION_FUNCTION_NAME("webstorePrivate.install");
+ DECLARE_EXTENSION_FUNCTION_NAME("webstorePrivate.completeInstall");
};
class GetBrowserLoginFunction : public SyncExtensionFunction {

Powered by Google App Engine
This is Rietveld 408576698