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

Unified Diff: trunk/src/chrome/browser/extensions/install_verifier.cc

Issue 147983011: Revert 247790 "Change default mode of extension install verifica..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 11 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: trunk/src/chrome/browser/extensions/install_verifier.cc
===================================================================
--- trunk/src/chrome/browser/extensions/install_verifier.cc (revision 247796)
+++ trunk/src/chrome/browser/extensions/install_verifier.cc (working copy)
@@ -30,11 +30,6 @@
NONE = 0, // Do not request install signatures, and do not enforce them.
BOOTSTRAP, // Request install signatures, but do not enforce them.
ENFORCE, // Request install signatures, and enforce them.
-
- // This is used in histograms - do not remove or reorder entries above! Also
- // the "MAX" item below should always be the last element.
-
- VERIFY_STATUS_MAX
};
#if defined(GOOGLE_CHROME_BUILD)
@@ -54,7 +49,7 @@
return ENFORCE;
}
- VerifyStatus default_status = NONE;
+ VerifyStatus default_status = BOOTSTRAP;
if (group == "Enforce")
return ENFORCE;
@@ -146,11 +141,6 @@
}
void InstallVerifier::Init() {
- UMA_HISTOGRAM_ENUMERATION("ExtensionInstallVerifier.ExperimentStatus",
- GetExperimentStatus(), VERIFY_STATUS_MAX);
- UMA_HISTOGRAM_ENUMERATION("ExtensionInstallVerifier.ActualStatus",
- GetStatus(), VERIFY_STATUS_MAX);
-
const base::DictionaryValue* pref = prefs_->GetInstallSignature();
if (pref) {
scoped_ptr<InstallSignature> signature_from_prefs =

Powered by Google App Engine
This is Rietveld 408576698