Remove RemoveChromeLegacyRegistryKeys() -- an old Registry cleanup function from Sept. 2009

chrome/installer/setup/uninstall.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // This file defines the methods useful for uninstalling Chrome.
 
 #include "chrome/installer/setup/uninstall.h"
 
 #include <vector>
 
@@ skipping 646 matching lines @@
   }
 
   // Note that we do not attempt to delete filetype associations since MSDN
   // says "Windows respects the Default value only if the ProgID found there is
   // a registered ProgID. If the ProgID is unregistered, it is ignored."
 
   *exit_code = installer::UNINSTALL_SUCCESSFUL;
   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
 // DelegateExecute data via an implementation of GetDelegateExecuteHandlerData.
 bool ProcessDelegateExecuteWorkItems(const InstallerState& installer_state,
                                      const Product& product) {
   scoped_ptr<WorkItemList> item_list(WorkItem::CreateNoRollbackWorkItemList());
   AddDelegateExecuteWorkItems(installer_state, FilePath(), Version(), product,
                               item_list.get());
   return item_list->Do();
@@ skipping 242 matching lines @@
 
   // Try and delete the preserved local state once the post-install
   // operations are complete.
   if (!backup_state_file.empty())
     file_util::Delete(backup_state_file, false);
 
   return ret;
 }
 
 }  // namespace installer