Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(188)

Side by Side Diff: chrome/browser/download/download_file_picker.h

Issue 1513413002: Enable "Hide Extension" option when "Save Link As" on the Mac Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix nits. Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_DOWNLOAD_DOWNLOAD_FILE_PICKER_H_ 5 #ifndef CHROME_BROWSER_DOWNLOAD_DOWNLOAD_FILE_PICKER_H_
6 #define CHROME_BROWSER_DOWNLOAD_DOWNLOAD_FILE_PICKER_H_ 6 #define CHROME_BROWSER_DOWNLOAD_DOWNLOAD_FILE_PICKER_H_
7 7
8 #include "base/callback.h" 8 #include "base/callback.h"
9 #include "ui/shell_dialogs/select_file_dialog.h" 9 #include "ui/shell_dialogs/select_file_dialog.h"
10 10
11 namespace base { 11 namespace base {
12 class FilePath; 12 class FilePath;
13 } 13 }
14 14
15 namespace content { 15 namespace content {
16 class DownloadItem; 16 class DownloadItem;
17 class DownloadManager; 17 class DownloadManager;
18 class WebContents; 18 class WebContents;
19 } 19 }
20 20
21 // Handles showing a dialog to the user to ask for the filename for a download. 21 // Handles showing a dialog to the user to ask for the filename for a download.
22 class DownloadFilePicker : public ui::SelectFileDialog::Listener { 22 class DownloadFilePicker : public ui::SelectFileDialog::Listener {
23 public: 23 public:
24 // Callback used to pass the user selection back to the owner of this 24 // Callback used to pass the user selection back to the owner of this
25 // object. 25 // object.
26 // |virtual_path|: The path chosen by the user. If the user cancels the file 26 // |virtual_path|: The path chosen by the user. If the user cancels the file
27 // selection, then this parameter will be the empty path. On Chrome OS, 27 // selection, then this parameter will be the empty path. On Chrome OS,
28 // this path may contain virtual mount points if the user chose a virtual 28 // this path may contain virtual mount points if the user chose a virtual
29 // path (e.g. Google Drive). 29 // path (e.g. Google Drive).
30 typedef base::Callback<void(const base::FilePath& virtual_path)> 30 typedef base::Callback<void(const base::FilePath& virtual_path,
31 FileSelectedCallback; 31 bool hide_file_extension)> FileSelectedCallback;
32 32
33 // Display a file picker dialog for |item|. The |suggested_path| will be used 33 // Display a file picker dialog for |item|. The |suggested_path| will be used
34 // as the initial path displayed to the user. |callback| will always be 34 // as the initial path displayed to the user. |callback| will always be
35 // invoked even if |item| is destroyed prior to the file picker completing. 35 // invoked even if |item| is destroyed prior to the file picker completing.
36 static void ShowFilePicker(content::DownloadItem* item, 36 static void ShowFilePicker(content::DownloadItem* item,
37 const base::FilePath& suggested_path, 37 const base::FilePath& suggested_path,
38 const FileSelectedCallback& callback); 38 const FileSelectedCallback& callback);
39 39
40 private: 40 private:
41 DownloadFilePicker(content::DownloadItem* item, 41 DownloadFilePicker(content::DownloadItem* item,
42 const base::FilePath& suggested_path, 42 const base::FilePath& suggested_path,
43 const FileSelectedCallback& callback); 43 const FileSelectedCallback& callback);
44 ~DownloadFilePicker() override; 44 ~DownloadFilePicker() override;
45 45
46 // Runs |file_selected_callback_| with |virtual_path| and then deletes this 46 // Runs |file_selected_callback_| with |virtual_path| and then deletes this
47 // object. 47 // object.
48 void OnFileSelected(const base::FilePath& virtual_path); 48 void OnFileSelected(const base::FilePath& virtual_path,
49 bool hide_file_extension);
49 50
50 // SelectFileDialog::Listener implementation. 51 // SelectFileDialog::Listener implementation.
51 void FileSelected(const base::FilePath& path, 52 void FileSelected(const base::FilePath& path,
52 int index, 53 int index,
53 void* params) override; 54 void* params) override;
55 void FileSelectedWithExtraInfo(const ui::SelectedFileInfo& file,
56 int index,
57 void* params) override;
54 void FileSelectionCanceled(void* params) override; 58 void FileSelectionCanceled(void* params) override;
55 59
56 // Initially suggested path. 60 // Initially suggested path.
57 base::FilePath suggested_path_; 61 base::FilePath suggested_path_;
58 62
59 // Callback invoked when a file selection is complete. 63 // Callback invoked when a file selection is complete.
60 FileSelectedCallback file_selected_callback_; 64 FileSelectedCallback file_selected_callback_;
61 65
62 // For managing select file dialogs. 66 // For managing select file dialogs.
63 scoped_refptr<ui::SelectFileDialog> select_file_dialog_; 67 scoped_refptr<ui::SelectFileDialog> select_file_dialog_;
64 68
65 // True if UMA regarding on the result of the file selection should be 69 // True if UMA regarding on the result of the file selection should be
66 // recorded. 70 // recorded.
67 bool should_record_file_picker_result_; 71 bool should_record_file_picker_result_;
68 72
69 DISALLOW_COPY_AND_ASSIGN(DownloadFilePicker); 73 DISALLOW_COPY_AND_ASSIGN(DownloadFilePicker);
70 }; 74 };
71 75
72 #endif // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_FILE_PICKER_H_ 76 #endif // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_FILE_PICKER_H_
OLDNEW
« no previous file with comments | « chrome/browser/download/chrome_download_manager_delegate_unittest.cc ('k') | chrome/browser/download/download_file_picker.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698