Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(141)

Issue 2909693004: Clean up the unpack directory for CRLSet component. (Closed)

Created:
3 years, 7 months ago by Sorin Jianu
Modified:
3 years, 7 months ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/branch-heads/3071
Project:
chromium
Visibility:
Public.

Description

Clean up the unpack directory for CRLSet component. In the current implementation of the component updater, the component installer owns the unpack path of a component and it is responsible for deleting the unpack path before the execution flow returns to the component updater during the update of a component. For most components, this clean up is done in the implementation of the DefaultComponentInstaller class. However, CRLSet is one of the few components which is not reusing the DefaultComponentInstaller for its component installer. As a result, it is leaking the content of the unpack path after each update cycle. It is not clear when this regression has been introduced. BUG=725671 Review-Url: https://codereview.chromium.org/2906683002 Cr-Original-Commit-Position: refs/heads/master@{#474700} Review-Url: https://codereview.chromium.org/2909693004 . Cr-Commit-Position: refs/branch-heads/3071@{#707} Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641} Committed: https://chromium.googlesource.com/chromium/src/+/5f54ca6de628657215c329dbc6bcf2720aad25dd

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
M chrome/browser/net/crl_set_fetcher.cc View 1 chunk +3 lines, -1 line 0 comments Download

Messages

Total messages: 2 (1 generated)
Sorin Jianu
3 years, 7 months ago (2017-05-26 21:22:26 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
5f54ca6de628657215c329dbc6bcf2720aad25dd.

Powered by Google App Engine
This is Rietveld 408576698