diff options
| author | bauerb <bauerb@chromium.org> | 2015-02-04 17:09:10 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-02-05 01:09:55 +0000 |
| commit | 810e60f487982e8edc208b78b0b4b2b2a12a77a0 (patch) | |
| tree | f76a8cb2498a8e32804fd0a9eb04703ed7a84127 /components/component_updater/component_updater_service.cc | |
| parent | abfbde8049ea4726e3c750a15aad66c5ab9f5093 (diff) | |
| download | chromium_src-810e60f487982e8edc208b78b0b4b2b2a12a77a0.zip chromium_src-810e60f487982e8edc208b78b0b4b2b2a12a77a0.tar.gz chromium_src-810e60f487982e8edc208b78b0b4b2b2a12a77a0.tar.bz2 | |
Make ComponentInstaller refcounted.
Before this CL, component installers were leaked in almost all cases. If we allow uninstalling components (see https://codereview.chromium.org/879993005/), we need to fix those leaks.
BUG=436459
Review URL: https://codereview.chromium.org/897873002
Cr-Commit-Position: refs/heads/master@{#314701}
Diffstat (limited to 'components/component_updater/component_updater_service.cc')
| -rw-r--r-- | components/component_updater/component_updater_service.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/components/component_updater/component_updater_service.cc b/components/component_updater/component_updater_service.cc index 4b97bae..89195c4 100644 --- a/components/component_updater/component_updater_service.cc +++ b/components/component_updater/component_updater_service.cc @@ -108,7 +108,7 @@ class CrxUpdateService : public ComponentUpdateService, public OnDemandUpdater { // Context for a crx download url request. struct CRXContext { - ComponentInstaller* installer; + scoped_refptr<ComponentInstaller> installer; std::vector<uint8_t> pk_hash; std::string id; std::string fingerprint; |