summaryrefslogtreecommitdiffstats
path: root/chrome/browser/download/download_file.h
diff options
context:
space:
mode:
authorpaul@chromium.org <paul@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-08-18 22:29:58 +0000
committerpaul@chromium.org <paul@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-08-18 22:29:58 +0000
commit8b6ff019c982f3929cf09b7090c9ec38c2751fc1 (patch)
tree015a4b4a26fd5c8729fa1f3a3f57e25e8666e586 /chrome/browser/download/download_file.h
parent7c23cfc44c6fa5d224acd7cae942dc7310138c7b (diff)
downloadchromium_src-8b6ff019c982f3929cf09b7090c9ec38c2751fc1.zip
chromium_src-8b6ff019c982f3929cf09b7090c9ec38c2751fc1.tar.gz
chromium_src-8b6ff019c982f3929cf09b7090c9ec38c2751fc1.tar.bz2
Show or open downloaded items on the UI thread for Mac. This is
required because NSWorkspace, which is used for the open or show operation, must be called on the main thread. BUG=19447 (http://crbug.com/19447) TEST=Repro steps fully described in the bug report. Review URL: http://codereview.chromium.org/172074 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23667 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/download/download_file.h')
-rw-r--r--chrome/browser/download/download_file.h6
1 files changed, 6 insertions, 0 deletions
diff --git a/chrome/browser/download/download_file.h b/chrome/browser/download/download_file.h
index 9706d4a..3527131 100644
--- a/chrome/browser/download/download_file.h
+++ b/chrome/browser/download/download_file.h
@@ -201,12 +201,18 @@ class DownloadFileManager
void RemoveDownloadManager(DownloadManager* manager);
void RemoveDownload(int id, DownloadManager* manager);
+#if !defined(OS_MACOSX)
+ // The open and show methods run on the file thread, which does not work on
+ // Mac OS X (which uses the UI thread for opens).
+
// Handler for shell operations sent from the UI to the download thread.
void OnShowDownloadInShell(const FilePath& full_path);
+
// Handler to open or execute a downloaded file.
void OnOpenDownloadInShell(const FilePath& full_path,
const GURL& url,
gfx::NativeView parent_window);
+#endif
// The download manager has provided a final name for a download. Sent from
// the UI thread and run on the download thread.