New issue
Advanced search Search tips

Issue 726783 link

Starred by 1 user
Status: Fixed
Owner:
sorin@chromium.org
Closed: May 2017
Cc:
waff...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Delete the unpack path in the DefaultComponentInstaller

Project Member Reported by sorin@chromium.org, May 26 2017 
This is related to https://bugs.chromium.org/p/chromium/issues/detail?id=725671

We want to ensure that the files in the unpack path are not left behind after the install is called.

This change affects the DefaultComponentInstaller implementation only. There are some components which don't reuse the DefaultComponentInstaller. These components are being migrated to use DefaultComponentInstaller.
Project Member

Comment 1 by bugdroid1@chromium.org, May 26 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/316232ae41d37cfe518010237b3f11225740ae8a

commit 316232ae41d37cfe518010237b3f11225740ae8a
Author: Sorin Jianu <sorin@chromium.org>
Date: Fri May 26 21:30:07 2017

Delete the unpack path in the DefaultComponentInstaller.

BUG= 726783 

Change-Id: I0fa62c5f21d07788307ff181f880aba3c07fdba3
Reviewed-on: https://chromium-review.googlesource.com/517442
Commit-Queue: Sorin Jianu <sorin@chromium.org>
Reviewed-by: Sorin Jianu <sorin@chromium.org>
Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
Cr-Commit-Position: refs/heads/master@{#475137}
[modify] https://crrev.com/316232ae41d37cfe518010237b3f11225740ae8a/components/component_updater/default_component_installer.cc
[modify] https://crrev.com/316232ae41d37cfe518010237b3f11225740ae8a/components/component_updater/default_component_installer_unittest.cc

Comment 2 by sorin@chromium.org, May 31 2017

Labels: -Pri-3 Pri-2
Status: Fixed (was: Started)
►

Sign in to add a comment