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

Unified Diff: ui/base/dialogs/select_file_dialog_win.cc

Issue 10908262: [cros] disable GDrive option on certain file pickers (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: self review Created 8 years, 3 months 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 side-by-side diff with in-line comments
Download patch
Index: ui/base/dialogs/select_file_dialog_win.cc
diff --git a/ui/base/dialogs/select_file_dialog_win.cc b/ui/base/dialogs/select_file_dialog_win.cc
index 0fd90ef76a06f2e1d83305ccef0b72f82707b2b0..2a41a23920122cc5ef7db8b1551226dec4cd56e9 100644
--- a/ui/base/dialogs/select_file_dialog_win.cc
+++ b/ui/base/dialogs/select_file_dialog_win.cc
@@ -126,6 +126,9 @@ std::wstring FormatFilterForExtensions(
DCHECK(file_ext.size() >= ext_desc.size());
+ if (file_ext.empty())
+ include_all_files = true;
+
std::wstring result;
for (size_t i = 0; i < file_ext.size(); ++i) {
@@ -441,11 +444,8 @@ class SelectFileDialogImpl : public ui::SelectFileDialog,
ui_proxy(MessageLoopForUI::current()->message_loop_proxy()),
owner(owner),
params(params) {
- if (file_types) {
+ if (file_types)
this->file_types = *file_types;
- } else {
- this->file_types.include_all_files = true;
- }
}
SelectFileDialog::Type type;
std::wstring title;
« ui/base/dialogs/select_file_dialog_mac.mm ('K') | « ui/base/dialogs/select_file_dialog_mac.mm ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698