Revert thakis's changes to download_file.cc from r17595.

BUG=13991

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@18307 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 8 insertions, 15 deletions

diff --git a/chrome/browser/download/download_file.cc b/chrome/browser/download/download_file.cc
index 8731b53..5dca83e 100644
--- a/chrome/browser/download/download_file.cc
+++ b/chrome/browser/download/download_file.cc
@@ -536,23 +536,16 @@ void DownloadFileManager::OnShowDownloadInShell(const FilePath& full_path) {
 void DownloadFileManager::OnOpenDownloadInShell(const FilePath& full_path,
                                                 const GURL& url,
                                                 gfx::NativeView parent_window) {
+  DCHECK(MessageLoop::current() == file_loop_);
 #if defined(OS_WIN)
-    DCHECK(MessageLoop::current() == file_loop_);
-    if (NULL != parent_window) {
-      win_util::SaferOpenItemViaShell(parent_window, L"", full_path,
-                                      UTF8ToWide(url.spec()));
-    } else {
-      win_util::OpenItemViaShell(full_path);
-    }
-#elif defined(OS_MACOSX)
-  // Quarantine takes care of asking the user about dangerous files, so we can
-  // just open it.
-  platform_util::OpenItem(full_path);
-#else
-  // TODO(port) implement me. (Does Linux need to use a "safe" open, or can it
-  // just share the Mac call to platform_util?)
-  NOTIMPLEMENTED();
+  if (NULL != parent_window) {
+    win_util::SaferOpenItemViaShell(parent_window, L"", full_path,
+                                    UTF8ToWide(url.spec()));
+    return;
+  }
 #endif
+
+  platform_util::OpenItem(full_path);
 }
 
 // The DownloadManager in the UI thread has provided a final name for the