| commit | c641adda4d6f720ab016e7e2a52fde109109f57a | |
| author | binjin <binjin@chromium.org> | |
| Wed, 15 Oct 2014 16:20:45 +0000 (15 09:20 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Wed, 15 Oct 2014 16:21:02 +0000 (15 16:21 +0000) | ||
| tree | 2e5538807ac03bd2b5e50ee2048fb468a6bfa2cc | treesnapshot (tar.gz zip) |
| parent | 36b11b6f5e533b384f45baa0d7833f1ab2bf1d2c | commitdiff |
Fix a regression regarding off-store extensions in metrics
This CL fixes a regression introduced by crrev.com/293777.
InstallVerifier::AllowedByEnterprisePolicy() are intended to query the policy system whether an extension are explicitly allowed by enterprise or not. But crrev.com/293777 changed the behavior into querying it's allowed to install or not, which in most cases, are true. But the ExtensionMetricsProvider considers every extension explicitly allowed by enterprise policy as from webstore, thus every off-store extensions are marked as from webstore in metrics.
Fix the regression by restoring into the original behavior.
BUG=423705
Review URL: https://codereview.chromium.org/651253003
Cr-Commit-Position: refs/heads/master@{#299694}
This CL fixes a regression introduced by crrev.com/293777.
InstallVerifier::AllowedByEnterprisePolicy() are intended to query the policy system whether an extension are explicitly allowed by enterprise or not. But crrev.com/293777 changed the behavior into querying it's allowed to install or not, which in most cases, are true. But the ExtensionMetricsProvider considers every extension explicitly allowed by enterprise policy as from webstore, thus every off-store extensions are marked as from webstore in metrics.
Fix the regression by restoring into the original behavior.
BUG=423705
Review URL: https://codereview.chromium.org/651253003
Cr-Commit-Position: refs/heads/master@{#299694}