| Index: chrome/installer/setup/uninstall.cc
|
| ===================================================================
|
| --- chrome/installer/setup/uninstall.cc (revision 30960)
|
| +++ chrome/installer/setup/uninstall.cc (working copy)
|
| @@ -179,12 +179,13 @@
|
| // Obtain the location of the user profile data. Chrome Frame needs to
|
| // build this path manually since it doesn't use the Chrome default dir.
|
| FilePath user_local_state;
|
| -#if defined(CHROME_FRAME_BUILD)
|
| - bool got_local_state =
|
| - chrome::GetChromeFrameUserDataDirectory(&user_local_state);
|
| -#else // if !defined(CHROME_FRAME_BUILD)
|
| - bool got_local_state = chrome::GetDefaultUserDataDirectory(&user_local_state);
|
| -#endif
|
| + bool got_local_state = false;
|
| + if (InstallUtil::IsChromeFrameProcess()) {
|
| + got_local_state =
|
| + chrome::GetChromeFrameUserDataDirectory(&user_local_state);
|
| + } else {
|
| + got_local_state = chrome::GetDefaultUserDataDirectory(&user_local_state);
|
| + }
|
|
|
| // Move the browser's persisted local state
|
| if (got_local_state) {
|
| @@ -204,20 +205,20 @@
|
| LOG(INFO) << "Deleting install path " << install_path;
|
| if (!file_util::Delete(install_path, true)) {
|
| LOG(ERROR) << "Failed to delete folder (1st try): " << install_path;
|
| -#if defined(CHROME_FRAME_BUILD)
|
| - // We don't try killing Chrome processes for Chrome Frame builds since
|
| - // that is unlikely to help. Instead, schedule files for deletion and
|
| - // return a value that will trigger a reboot prompt.
|
| - ScheduleDirectoryForDeletion(install_path.c_str());
|
| - result = DELETE_REQUIRES_REBOOT;
|
| -#else
|
| - // Try closing any running chrome processes and deleting files once again.
|
| - CloseAllChromeProcesses();
|
| - if (!file_util::Delete(install_path, true)) {
|
| - LOG(ERROR) << "Failed to delete folder (2nd try): " << install_path;
|
| - result = DELETE_FAILED;
|
| + if (InstallUtil::IsChromeFrameProcess()) {
|
| + // We don't try killing Chrome processes for Chrome Frame builds since
|
| + // that is unlikely to help. Instead, schedule files for deletion and
|
| + // return a value that will trigger a reboot prompt.
|
| + ScheduleDirectoryForDeletion(install_path.c_str());
|
| + result = DELETE_REQUIRES_REBOOT;
|
| + } else {
|
| + // Try closing any running chrome processes and deleting files once again.
|
| + CloseAllChromeProcesses();
|
| + if (!file_util::Delete(install_path, true)) {
|
| + LOG(ERROR) << "Failed to delete folder (2nd try): " << install_path;
|
| + result = DELETE_FAILED;
|
| + }
|
| }
|
| -#endif
|
| }
|
|
|
| if (delete_profile && got_local_state) {
|
| @@ -225,12 +226,12 @@
|
| if (!file_util::Delete(user_local_state, true)) {
|
| LOG(ERROR) << "Failed to delete user profile dir: "
|
| << user_local_state.value();
|
| -#if defined(CHROME_FRAME_BUILD)
|
| - ScheduleDirectoryForDeletion(user_local_state.value().c_str());
|
| - result = DELETE_REQUIRES_REBOOT;
|
| -#else
|
| - result = DELETE_FAILED;
|
| -#endif
|
| + if (InstallUtil::IsChromeFrameProcess()) {
|
| + ScheduleDirectoryForDeletion(user_local_state.value().c_str());
|
| + result = DELETE_REQUIRES_REBOOT;
|
| + } else {
|
| + result = DELETE_FAILED;
|
| + }
|
| }
|
| DeleteEmptyParentDir(user_local_state.value());
|
| }
|
| @@ -407,9 +408,8 @@
|
| if (force_uninstall) {
|
| // Since --force-uninstall command line option is used, we are going to
|
| // do silent uninstall. Try to close all running Chrome instances.
|
| -#if !defined(CHROME_FRAME_BUILD)
|
| - CloseAllChromeProcesses();
|
| -#endif
|
| + if (!InstallUtil::IsChromeFrameProcess())
|
| + CloseAllChromeProcesses();
|
| } else { // no --force-uninstall so lets show some UI dialog boxes.
|
| status = IsChromeActiveOrUserCancelled(system_uninstall);
|
| if (status != installer_util::UNINSTALL_CONFIRMED &&
|
|
|