diff options
| author | mpcomplete@chromium.org <mpcomplete@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-08-17 13:33:16 +0000 |
|---|---|---|
| committer | mpcomplete@chromium.org <mpcomplete@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-08-17 13:33:16 +0000 |
| commit | 3e3fafbb893fe00a959dc722e80b3147be770878 (patch) | |
| tree | b8b272796467843aa1dfc776bc056edb970e67c5 /chrome/browser/extensions | |
| parent | 0b1b283b32bd7f893782e6f56330a07b0ee988a5 (diff) | |
| download | chromium_src-3e3fafbb893fe00a959dc722e80b3147be770878.zip chromium_src-3e3fafbb893fe00a959dc722e80b3147be770878.tar.gz chromium_src-3e3fafbb893fe00a959dc722e80b3147be770878.tar.bz2 | |
Don't report Extensions.ExternalExtensionEvent.INSTALLED for autoupdates.
Generally we wouldn't reach that code for an autoupdate, because ack_external would already have been set. However, it's possible the extension updates before the user notices the prompt, which would result in this case.
BUG=268666
Review URL: https://chromiumcodereview.appspot.com/22877021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218165 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions')
| -rw-r--r-- | chrome/browser/extensions/extension_service.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/extensions/extension_service.cc b/chrome/browser/extensions/extension_service.cc index 6300e82..ed7d9ce 100644 --- a/chrome/browser/extensions/extension_service.cc +++ b/chrome/browser/extensions/extension_service.cc @@ -2612,7 +2612,7 @@ void ExtensionService::FinishInstallation(const Extension* extension) { // If this is a new external extension that was disabled, alert the user // so he can reenable it. We do this last so that it has already been // added to our list of extensions. - if (unacknowledged_external) { + if (unacknowledged_external && !is_update) { UpdateExternalExtensionAlert(); UMA_HISTOGRAM_ENUMERATION("Extensions.ExternalExtensionEvent", EXTERNAL_EXTENSION_INSTALLED, |