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

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

Issue 10454067: Remove RemoveChromeLegacyRegistryKeys() -- an old Registry cleanup function from Sept. 2009 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 7 months 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
« no previous file with comments | « chrome/installer/setup/setup_main.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/installer/setup/uninstall.cc
diff --git a/chrome/installer/setup/uninstall.cc b/chrome/installer/setup/uninstall.cc
index 86b374fa710dcfb48b072e5f3ab8fa08161bb737..c0f7ea07fedafd57786c713d8e345d883205e9d4 100644
--- a/chrome/installer/setup/uninstall.cc
+++ b/chrome/installer/setup/uninstall.cc
@@ -664,39 +664,6 @@ bool DeleteChromeRegistrationKeys(BrowserDistribution* dist, HKEY root,
return true;
}
-void RemoveChromeLegacyRegistryKeys(BrowserDistribution* dist) {
- // We used to register Chrome to handle crx files, but this turned out
- // to be not worth the hassle. Remove these old registry entries if
- // they exist. See: http://codereview.chromium.org/210007
-
-#if defined(GOOGLE_CHROME_BUILD)
-const wchar_t kChromeExtProgId[] = L"ChromeExt";
-#else
-const wchar_t kChromeExtProgId[] = L"ChromiumExt";
-#endif
-
- HKEY roots[] = { HKEY_LOCAL_MACHINE, HKEY_CURRENT_USER };
- for (size_t i = 0; i < arraysize(roots); ++i) {
- string16 suffix;
- if (roots[i] == HKEY_LOCAL_MACHINE &&
- !ShellUtil::GetUserSpecificDefaultBrowserSuffix(dist, &suffix))
- suffix = L"";
-
- // Delete Software\Classes\ChromeExt,
- string16 ext_prog_id(ShellUtil::kRegClasses);
- ext_prog_id.push_back(FilePath::kSeparators[0]);
- ext_prog_id.append(kChromeExtProgId);
- ext_prog_id.append(suffix);
- InstallUtil::DeleteRegistryKey(roots[i], ext_prog_id);
-
- // Delete Software\Classes\.crx,
- string16 ext_association(ShellUtil::kRegClasses);
- ext_association.append(L"\\");
- ext_association.append(chrome::kExtensionFileExtension);
- InstallUtil::DeleteRegistryKey(roots[i], ext_association);
- }
-}
-
// Builds and executes a work item list to remove DelegateExecute verb handler
// work items for |product|. This will be a noop for products whose
// corresponding BrowserDistribution implementations do not publish
« no previous file with comments | « chrome/installer/setup/setup_main.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698