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

Unified Diff: chrome/installer/setup/uninstall.cc

Issue 2507753002: Install the chrome event log provider together with the browser. (Closed)
Patch Set: Fix RegisterEventLogProvider unit test. Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/installer/setup/uninstall.cc
diff --git a/chrome/installer/setup/uninstall.cc b/chrome/installer/setup/uninstall.cc
index 3365f2a0b7bb852b8270d081a74cb0d67bc125bf..66717708e20d25eaaa129914ec16dd207039e0d1 100644
--- a/chrome/installer/setup/uninstall.cc
+++ b/chrome/installer/setup/uninstall.cc
@@ -1230,6 +1230,10 @@ InstallStatus UninstallProduct(const InstallationState& original_state,
reg_path.append(installer::kChromeExe);
InstallUtil::DeleteRegistryKey(HKEY_LOCAL_MACHINE, reg_path,
WorkItem::kWow64Default);
+
+ // Remove the event log provider registration as we are going to delete
+ // the file which serves the resources anyways.
+ DeRegisterEventLogProvider();
grt (UTC plus 2) 2017/01/09 11:41:52 i just noticed that this shouldn't be within the "
}
// Unregister any dll servers that we may have registered for this
« no previous file with comments | « chrome/installer/setup/setup_util_unittest.cc ('k') | chrome/test/mini_installer/config/chrome_system_installed.prop » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698