Index: chrome/browser/download/save_package.cc |
=================================================================== |
--- chrome/browser/download/save_package.cc (revision 9983) |
+++ chrome/browser/download/save_package.cc (working copy) |
@@ -961,19 +961,12 @@ |
// Since we take the suggested name from the web page's title, we want to |
// ignore the file extension generated by SaveFileAsWithFilter, since it |
// will always be ".htm". |
- // TODO(estade): is this saved_main_file_path assignment behavior desired? |
- // It was copied from previous code but seems strange. |
std::wstring main_file_path; |
- if (!win_util::SaveFileAsWithFilter(container_hwnd, |
- suggest_name.value(), |
- filter, |
- L"htm", |
- true, |
- &index, |
- &main_file_path)) { |
- param->saved_main_file_path = FilePath(main_file_path); |
+ bool success = win_util::SaveFileAsWithFilter(container_hwnd, |
+ suggest_name.value(), filter, L"htm", true, &index, &main_file_path); |
+ param->saved_main_file_path = FilePath(main_file_path); |
+ if (!success) |
return false; |
- } |
} else { |
param->saved_main_file_path = suggest_name; |
} |