Unconditionally delete the unpack path in the UpdateClient. |
||||||
Issue descriptionDoing so, enforces that temporary files are not left behind after unpacking, even if installing of a CRX fails.
,
May 31 2017
,
May 31 2017
,
May 31 2017
Your change meets the bar and is auto-approved for M60. Please go ahead and merge the CL to branch 3112 manually. Please contact milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 31 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/993e502cb59a07df852caf1d6c940c998b71f04f commit 993e502cb59a07df852caf1d6c940c998b71f04f Author: Sorin Jianu <sorin@chromium.org> Date: Wed May 31 21:55:03 2017 Unconditionally delete the unpack path in the UpdateClient. Bug: 726959 Change-Id: I002aa30ee0d85d845dba915d9dbd6e5599218184 Reviewed-on: https://chromium-review.googlesource.com/517463 Reviewed-by: Joshua Pawlicki <waffles@chromium.org> Commit-Queue: Sorin Jianu <sorin@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#475648} Review-Url: https://codereview.chromium.org/2918653003 . Cr-Commit-Position: refs/branch-heads/3112@{#65} Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897} [modify] https://crrev.com/993e502cb59a07df852caf1d6c940c998b71f04f/components/update_client/component.cc [modify] https://crrev.com/993e502cb59a07df852caf1d6c940c998b71f04f/components/update_client/test_installer.cc [modify] https://crrev.com/993e502cb59a07df852caf1d6c940c998b71f04f/components/update_client/test_installer.h [modify] https://crrev.com/993e502cb59a07df852caf1d6c940c998b71f04f/components/update_client/update_client_unittest.cc
,
Jun 1 2017
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by bugdroid1@chromium.org
, May 30 2017