| 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;
|
| }
|
|
|