New issue
Advanced search Search tips

Issue 679707 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

Put eventlog_provider.dll uninstall step in the right place.

Project Member Reported by pastarmovj@chromium.org, Jan 10 2017

Issue description

It was incorrectly put in a conditional statement that just happened to be always true in the right situation but is not guaranteed to always be this way.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 10 2017

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

commit aa4e14837187e3c65d9175961d6e8ce3ca013f3c
Author: pastarmovj <pastarmovj@chromium.org>
Date: Tue Jan 10 12:50:35 2017

Move eventlog_provider.dll uninstall code to its proper location.

It was incorrectly put in a conditional statement that just happened
to be always true in the right situation but is not guaranteed to
always be this way.

BUG= 679707 

Review-Url: https://codereview.chromium.org/2621933002
Cr-Commit-Position: refs/heads/master@{#442562}

[modify] https://crrev.com/aa4e14837187e3c65d9175961d6e8ce3ca013f3c/chrome/installer/setup/uninstall.cc

Status: Fixed (was: Started)

Sign in to add a comment