Chromium Code Reviews| 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); |
| }; |