Index: chrome/browser/download/download_file.cc |
=================================================================== |
--- chrome/browser/download/download_file.cc (revision 17020) |
+++ chrome/browser/download/download_file.cc (working copy) |
@@ -530,24 +530,23 @@ |
platform_util::ShowItemInFolder(full_path); |
} |
-// Launches the selected download using ShellExecute 'open' verb. If there is |
-// a valid parent window, the 'safer' version will be used which can |
+// Launches the selected download using ShellExecute 'open' verb. For windows, |
+// if there is a valid parent window, the 'safer' version will be used which can |
// display a modal dialog asking for user consent on dangerous files. |
void DownloadFileManager::OnOpenDownloadInShell(const FilePath& full_path, |
const GURL& url, |
gfx::NativeView parent_window) { |
-#if defined(OS_WIN) |
DCHECK(MessageLoop::current() == file_loop_); |
+ |
+#if defined(OS_WIN) |
if (NULL != parent_window) { |
win_util::SaferOpenItemViaShell(parent_window, L"", full_path, |
- UTF8ToWide(url.spec()), true); |
- } else { |
- win_util::OpenItemViaShell(full_path, true); |
+ UTF8ToWide(url.spec())); |
+ return; |
} |
-#else |
- // TODO(port) implement me. |
- NOTREACHED(); |
#endif |
+ |
+ platform_util::OpenItem(full_path); |
} |
// The DownloadManager in the UI thread has provided a final name for the |