Commit 32a5d0dd authored by grt@chromium.org's avatar grt@chromium.org

Don't try to delete file type ownership at uninstall.

As per http://msdn.microsoft.com/en-us/library/windows/desktop/cc144148(v=vs.85).aspx#uninstall

BUG=103935
TEST=install Chrome, make it the default, then uninstall it.  make sure HKLM\Software\Classes\.htm hasn't been deleted.


Review URL: http://codereview.chromium.org/8536023

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109698 0039d316-1c4b-4281-b951-d872f2087c98
parent 8336aaf4
...@@ -623,16 +623,9 @@ bool DeleteChromeRegistrationKeys(BrowserDistribution* dist, HKEY root, ...@@ -623,16 +623,9 @@ bool DeleteChromeRegistrationKeys(BrowserDistribution* dist, HKEY root,
open_command_pred); open_command_pred);
} }
// Delete each filetype association if it references this Chrome. // Note that we do not attempt to delete filetype associations since MSDN
InstallUtil::ValueEquals prog_id_pred(ShellUtil::kChromeHTMLProgId + // says "Windows respects the Default value only if the ProgID found there is
browser_entry_suffix); // a registered ProgID. If the ProgID is unregistered, it is ignored."
for (const wchar_t* const* filetype = &ShellUtil::kFileAssociations[0];
*filetype != NULL; ++filetype) {
parent_key.resize(base_length);
file_util::AppendToPath(&parent_key, *filetype);
InstallUtil::DeleteRegistryKeyIf(root, parent_key, parent_key, L"",
prog_id_pred);
}
*exit_code = installer::UNINSTALL_SUCCESSFUL; *exit_code = installer::UNINSTALL_SUCCESSFUL;
return true; return true;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment