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

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

Issue 10542048: Add a group policy controlling which sites can install extensions. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: cleanup Created 8 years, 6 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/crx_installer.h
diff --git a/chrome/browser/extensions/crx_installer.h b/chrome/browser/extensions/crx_installer.h
index 5a9d51b4d25a8d51fbb3df11d9b5f56c86b5cc1e..f239de684b6a24e2110591ed456fec1972caad65 100644
--- a/chrome/browser/extensions/crx_installer.h
+++ b/chrome/browser/extensions/crx_installer.h
@@ -54,6 +54,14 @@ class CrxInstaller
: public SandboxedExtensionUnpackerClient,
public ExtensionInstallUI::Delegate {
public:
+ // Used in histograms; do not change order.
+ enum OffStoreInstallAllowReason {
+ OffStoreInstallDisallowed,
+ OffStoreInstallAllowedFromSettingsPage,
+ OffStoreInstallAllowedBecausePref,
+ NumOffStoreInstallAllowReasons
+ };
+
// Extensions will be installed into frontend->install_directory(),
// then registered with |frontend|. Any install UI will be displayed
// using |client|. Pass NULL for |client| for silent install
@@ -146,8 +154,12 @@ class CrxInstaller
install_cause_ = install_cause;
}
- bool allow_off_store_install() const { return allow_off_store_install_; }
- void set_allow_off_store_install(bool val) { allow_off_store_install_ = val; }
Yoyo Zhou 2012/06/07 19:19:15 You need to change all the other callers of this (
Aaron Boodman 2012/06/07 22:37:11 Fussy, fussy.
+ OffStoreInstallAllowReason off_store_install_allow_reason() const {
+ return off_store_install_allow_reason_;
+ }
+ void set_off_store_install_allow_reason(OffStoreInstallAllowReason reason) {
+ off_store_install_allow_reason_ = reason;
+ }
void set_page_ordinal(const StringOrdinal& page_ordinal) {
page_ordinal_ = page_ordinal;
@@ -312,7 +324,7 @@ class CrxInstaller
int creation_flags_;
// Whether to allow off store installation.
- bool allow_off_store_install_;
+ OffStoreInstallAllowReason off_store_install_allow_reason_;
DISALLOW_COPY_AND_ASSIGN(CrxInstaller);
};

Powered by Google App Engine
This is Rietveld 408576698