2022-04-15 21:55:23 +02:00
|
|
|
diff --git chrome/browser/file_select_helper.cc chrome/browser/file_select_helper.cc
|
2024-05-29 22:38:00 +02:00
|
|
|
index 447a91b9ac380..0452f0a8b57cf 100644
|
2022-04-15 21:55:23 +02:00
|
|
|
--- chrome/browser/file_select_helper.cc
|
|
|
|
+++ chrome/browser/file_select_helper.cc
|
|
|
|
@@ -20,6 +20,7 @@
|
|
|
|
#include "base/threading/hang_watcher.h"
|
|
|
|
#include "build/build_config.h"
|
|
|
|
#include "build/chromeos_buildflags.h"
|
|
|
|
+#include "cef/libcef/features/runtime.h"
|
|
|
|
#include "chrome/browser/browser_process.h"
|
2023-02-27 19:52:38 +01:00
|
|
|
#include "chrome/browser/enterprise/connectors/common.h"
|
|
|
|
#include "chrome/browser/platform_util.h"
|
2024-04-30 03:09:17 +02:00
|
|
|
@@ -245,6 +246,17 @@ void FileSelectHelper::OnListFile(
|
2022-04-15 21:55:23 +02:00
|
|
|
void FileSelectHelper::LaunchConfirmationDialog(
|
|
|
|
const base::FilePath& path,
|
|
|
|
std::vector<ui::SelectedFileInfo> selected_files) {
|
2024-04-30 03:09:17 +02:00
|
|
|
+ if (
|
|
|
|
+#if BUILDFLAG(ENABLE_ALLOY_BOOTSTRAP)
|
|
|
|
+ cef::IsAlloyRuntimeEnabled() ||
|
|
|
|
+#endif
|
|
|
|
+ run_from_cef_) {
|
2022-04-15 21:55:23 +02:00
|
|
|
+ // Don't show the upload confirmation dialog with the Alloy runtime, or
|
|
|
|
+ // when triggered via CEF (initially or recursively).
|
|
|
|
+ ConvertToFileChooserFileInfoList(selected_files);
|
|
|
|
+ return;
|
|
|
|
+ }
|
|
|
|
+
|
|
|
|
ShowFolderUploadConfirmationDialog(
|
|
|
|
path,
|
|
|
|
base::BindOnce(&FileSelectHelper::ConvertToFileChooserFileInfoList, this),
|
2024-04-30 03:09:17 +02:00
|
|
|
@@ -329,6 +341,14 @@ void FileSelectHelper::PerformContentAnalysisIfNeeded(
|
2022-09-30 20:09:37 +02:00
|
|
|
if (AbortIfWebContentsDestroyed())
|
|
|
|
return;
|
|
|
|
|
2024-04-30 03:09:17 +02:00
|
|
|
+#if BUILDFLAG(ENABLE_ALLOY_BOOTSTRAP)
|
2022-09-30 20:09:37 +02:00
|
|
|
+ // Don't trigger creation of a AccountConsistencyModeManager (see issue #3401)
|
|
|
|
+ if (cef::IsAlloyRuntimeEnabled()) {
|
|
|
|
+ NotifyListenerAndEnd(std::move(list));
|
|
|
|
+ return;
|
|
|
|
+ }
|
2024-04-30 03:09:17 +02:00
|
|
|
+#endif
|
2022-09-30 20:09:37 +02:00
|
|
|
+
|
2023-07-21 00:11:34 +02:00
|
|
|
#if BUILDFLAG(ENTERPRISE_CLOUD_CONTENT_ANALYSIS)
|
2022-09-30 20:09:37 +02:00
|
|
|
enterprise_connectors::ContentAnalysisDelegate::Data data;
|
|
|
|
if (enterprise_connectors::ContentAnalysisDelegate::IsEnabled(
|
2024-05-29 22:38:00 +02:00
|
|
|
@@ -472,31 +492,51 @@ FileSelectHelper::GetFileTypesFromAcceptType(
|
2022-04-15 21:55:23 +02:00
|
|
|
std::vector<base::FilePath::StringType>* extensions =
|
|
|
|
&file_type->extensions.back();
|
|
|
|
|
|
|
|
+ // Create individual filters for each accept type.
|
|
|
|
+ std::vector<std::vector<base::FilePath::StringType>> all_extensions;
|
|
|
|
+ std::vector<std::u16string> all_overrides;
|
2024-05-29 22:38:00 +02:00
|
|
|
+ std::vector<std::u16string> all_mimetypes;
|
2022-04-15 21:55:23 +02:00
|
|
|
+
|
|
|
|
// Find the corresponding extensions.
|
|
|
|
int valid_type_count = 0;
|
|
|
|
int description_id = 0;
|
2024-05-29 22:38:00 +02:00
|
|
|
+ std::string ascii_type;
|
2022-04-15 21:55:23 +02:00
|
|
|
for (const auto& accept_type : accept_types) {
|
|
|
|
+ std::vector<base::FilePath::StringType> current_extensions;
|
|
|
|
+ description_id = 0;
|
2024-05-29 22:38:00 +02:00
|
|
|
+ ascii_type.clear();
|
2022-04-15 21:55:23 +02:00
|
|
|
+
|
|
|
|
size_t old_extension_size = extensions->size();
|
|
|
|
if (accept_type[0] == '.') {
|
|
|
|
// If the type starts with a period it is assumed to be a file extension
|
|
|
|
// so we just have to add it to the list.
|
|
|
|
base::FilePath::StringType ext =
|
|
|
|
base::FilePath::FromUTF16Unsafe(accept_type).value();
|
|
|
|
- extensions->push_back(ext.substr(1));
|
|
|
|
+ current_extensions.push_back(ext.substr(1));
|
|
|
|
} else {
|
|
|
|
if (!base::IsStringASCII(accept_type))
|
|
|
|
continue;
|
2024-05-29 22:38:00 +02:00
|
|
|
- std::string ascii_type = base::UTF16ToASCII(accept_type);
|
|
|
|
+ ascii_type = base::UTF16ToASCII(accept_type);
|
|
|
|
if (ascii_type == "image/*")
|
|
|
|
description_id = IDS_IMAGE_FILES;
|
|
|
|
else if (ascii_type == "audio/*")
|
2022-04-15 21:55:23 +02:00
|
|
|
description_id = IDS_AUDIO_FILES;
|
|
|
|
else if (ascii_type == "video/*")
|
|
|
|
description_id = IDS_VIDEO_FILES;
|
|
|
|
-
|
|
|
|
- net::GetExtensionsForMimeType(ascii_type, extensions);
|
|
|
|
+ net::GetExtensionsForMimeType(ascii_type, ¤t_extensions);
|
|
|
|
}
|
|
|
|
|
|
|
|
+ if (!current_extensions.empty()) {
|
|
|
|
+ all_extensions.push_back(current_extensions);
|
|
|
|
+ all_overrides.push_back(description_id != 0 ?
|
|
|
|
+ l10n_util::GetStringUTF16(description_id) :
|
|
|
|
+ std::u16string());
|
2024-05-29 22:38:00 +02:00
|
|
|
+ all_mimetypes.push_back(ascii_type.empty() ?
|
|
|
|
+ std::u16string() : accept_type);
|
2022-04-15 21:55:23 +02:00
|
|
|
+
|
|
|
|
+ extensions->insert(extensions->end(), current_extensions.begin(),
|
|
|
|
+ current_extensions.end());
|
|
|
|
+ }
|
|
|
|
if (extensions->size() > old_extension_size)
|
|
|
|
valid_type_count++;
|
|
|
|
}
|
2024-05-29 22:38:00 +02:00
|
|
|
@@ -513,12 +553,28 @@ FileSelectHelper::GetFileTypesFromAcceptType(
|
|
|
|
// dialog uses the first extension in the list to form the description,
|
|
|
|
// like "EHTML Files". This is not what we want.
|
|
|
|
if (valid_type_count > 1 ||
|
|
|
|
- (valid_type_count == 1 && description_id == 0 && extensions->size() > 1))
|
|
|
|
+ (valid_type_count == 1 && description_id == 0 && extensions->size() > 1)) {
|
|
|
|
description_id = IDS_CUSTOM_FILES;
|
|
|
|
+ ascii_type.clear();
|
|
|
|
+ }
|
2022-04-15 21:55:23 +02:00
|
|
|
|
2024-05-29 22:38:00 +02:00
|
|
|
- if (description_id) {
|
|
|
|
- file_type->extension_description_overrides.push_back(
|
|
|
|
- l10n_util::GetStringUTF16(description_id));
|
|
|
|
+ file_type->extension_description_overrides.push_back(
|
|
|
|
+ description_id != 0 ?
|
|
|
|
+ l10n_util::GetStringUTF16(description_id) :
|
|
|
|
+ std::u16string());
|
|
|
|
+ file_type->extension_mimetypes.push_back(
|
|
|
|
+ ascii_type.empty() ? std::u16string() : base::ASCIIToUTF16(ascii_type));
|
|
|
|
+
|
|
|
|
+ if (all_extensions.size() > 1) {
|
2022-04-15 21:55:23 +02:00
|
|
|
+ // Insert filters for the specific accept types at the beginning.
|
|
|
|
+ file_type->extensions.insert(file_type->extensions.begin(),
|
|
|
|
+ all_extensions.begin(), all_extensions.end());
|
|
|
|
+ file_type->extension_description_overrides.insert(
|
|
|
|
+ file_type->extension_description_overrides.begin(),
|
|
|
|
+ all_overrides.begin(), all_overrides.end());
|
2024-05-29 22:38:00 +02:00
|
|
|
+ file_type->extension_mimetypes.insert(
|
|
|
|
+ file_type->extension_mimetypes.begin(),
|
|
|
|
+ all_mimetypes.begin(), all_mimetypes.end());
|
|
|
|
}
|
2022-04-15 21:55:23 +02:00
|
|
|
|
2024-05-29 22:38:00 +02:00
|
|
|
return file_type;
|
|
|
|
@@ -528,7 +584,8 @@ FileSelectHelper::GetFileTypesFromAcceptType(
|
2022-04-15 21:55:23 +02:00
|
|
|
void FileSelectHelper::RunFileChooser(
|
|
|
|
content::RenderFrameHost* render_frame_host,
|
|
|
|
scoped_refptr<content::FileSelectListener> listener,
|
|
|
|
- const FileChooserParams& params) {
|
|
|
|
+ const FileChooserParams& params,
|
|
|
|
+ bool run_from_cef) {
|
|
|
|
Profile* profile = Profile::FromBrowserContext(
|
|
|
|
render_frame_host->GetProcess()->GetBrowserContext());
|
|
|
|
|
2024-05-29 22:38:00 +02:00
|
|
|
@@ -547,6 +604,7 @@ void FileSelectHelper::RunFileChooser(
|
2022-04-15 21:55:23 +02:00
|
|
|
// message.
|
|
|
|
scoped_refptr<FileSelectHelper> file_select_helper(
|
|
|
|
new FileSelectHelper(profile));
|
|
|
|
+ file_select_helper->run_from_cef_ = run_from_cef;
|
|
|
|
file_select_helper->RunFileChooser(render_frame_host, std::move(listener),
|
|
|
|
params.Clone());
|
|
|
|
}
|
|
|
|
diff --git chrome/browser/file_select_helper.h chrome/browser/file_select_helper.h
|
2024-05-29 22:38:00 +02:00
|
|
|
index 7194323c36956..fd4a5361798f0 100644
|
2022-04-15 21:55:23 +02:00
|
|
|
--- chrome/browser/file_select_helper.h
|
|
|
|
+++ chrome/browser/file_select_helper.h
|
2023-07-21 00:11:34 +02:00
|
|
|
@@ -62,7 +62,8 @@ class FileSelectHelper : public base::RefCountedThreadSafe<
|
2022-04-15 21:55:23 +02:00
|
|
|
static void RunFileChooser(
|
|
|
|
content::RenderFrameHost* render_frame_host,
|
|
|
|
scoped_refptr<content::FileSelectListener> listener,
|
|
|
|
- const blink::mojom::FileChooserParams& params);
|
|
|
|
+ const blink::mojom::FileChooserParams& params,
|
|
|
|
+ bool run_from_cef = false);
|
|
|
|
|
|
|
|
// Enumerates all the files in directory.
|
|
|
|
static void EnumerateDirectory(
|
2024-05-29 22:38:00 +02:00
|
|
|
@@ -325,6 +326,9 @@ class FileSelectHelper : public base::RefCountedThreadSafe<
|
2022-04-15 21:55:23 +02:00
|
|
|
// Set to false in unit tests since there is no WebContents.
|
|
|
|
bool abort_on_missing_web_contents_in_tests_ = true;
|
2022-07-21 19:26:10 +02:00
|
|
|
|
2022-04-15 21:55:23 +02:00
|
|
|
+ // Set to true if this dialog was triggered via CEF.
|
|
|
|
+ bool run_from_cef_ = false;
|
2022-07-21 19:26:10 +02:00
|
|
|
+
|
|
|
|
#if BUILDFLAG(IS_CHROMEOS_ASH)
|
2022-09-26 21:30:45 +02:00
|
|
|
base::WeakPtrFactory<FileSelectHelper> weak_ptr_factory_{this};
|
|
|
|
#endif // BUILDFLAG(IS_CHROMEOS_ASH)
|
2022-04-15 21:55:23 +02:00
|
|
|
diff --git chrome/browser/ui/chrome_select_file_policy.h chrome/browser/ui/chrome_select_file_policy.h
|
2023-08-09 23:17:17 +02:00
|
|
|
index 49272553c7c53..5ba90c9a06ecf 100644
|
2022-04-15 21:55:23 +02:00
|
|
|
--- chrome/browser/ui/chrome_select_file_policy.h
|
|
|
|
+++ chrome/browser/ui/chrome_select_file_policy.h
|
|
|
|
@@ -30,6 +30,8 @@ class ChromeSelectFilePolicy : public ui::SelectFilePolicy {
|
|
|
|
// Returns true if local state allows showing file pickers.
|
|
|
|
static bool FileSelectDialogsAllowed();
|
|
|
|
|
|
|
|
+ content::WebContents* source_contents() const { return source_contents_; }
|
|
|
|
+
|
|
|
|
private:
|
2023-08-09 23:17:17 +02:00
|
|
|
raw_ptr<content::WebContents, AcrossTasksDanglingUntriaged> source_contents_;
|
2022-04-15 21:55:23 +02:00
|
|
|
};
|
|
|
|
diff --git ui/shell_dialogs/execute_select_file_win.cc ui/shell_dialogs/execute_select_file_win.cc
|
2023-11-21 20:17:55 +01:00
|
|
|
index 9361287ac3411..59567b485bf99 100644
|
2022-04-15 21:55:23 +02:00
|
|
|
--- ui/shell_dialogs/execute_select_file_win.cc
|
|
|
|
+++ ui/shell_dialogs/execute_select_file_win.cc
|
2023-11-21 20:17:55 +01:00
|
|
|
@@ -360,9 +360,7 @@ bool ExecuteSelectSingleFile(HWND owner,
|
2022-04-15 21:55:23 +02:00
|
|
|
const std::vector<FileFilterSpec>& filter,
|
|
|
|
int* filter_index,
|
|
|
|
std::vector<base::FilePath>* paths) {
|
|
|
|
- // Note: The title is not passed down for historical reasons.
|
|
|
|
- // TODO(pmonette): Figure out if it's a worthwhile improvement.
|
|
|
|
- return RunOpenFileDialog(owner, std::u16string(), std::u16string(),
|
|
|
|
+ return RunOpenFileDialog(owner, title, std::u16string(),
|
|
|
|
default_path, filter, 0, filter_index, paths);
|
|
|
|
}
|
|
|
|
|
2023-11-21 20:17:55 +01:00
|
|
|
@@ -374,14 +372,13 @@ bool ExecuteSelectMultipleFile(HWND owner,
|
2022-04-15 21:55:23 +02:00
|
|
|
std::vector<base::FilePath>* paths) {
|
|
|
|
DWORD dialog_options = FOS_ALLOWMULTISELECT;
|
|
|
|
|
|
|
|
- // Note: The title is not passed down for historical reasons.
|
|
|
|
- // TODO(pmonette): Figure out if it's a worthwhile improvement.
|
|
|
|
- return RunOpenFileDialog(owner, std::u16string(), std::u16string(),
|
|
|
|
+ return RunOpenFileDialog(owner, title, std::u16string(),
|
|
|
|
default_path, filter, dialog_options, filter_index,
|
|
|
|
paths);
|
|
|
|
}
|
|
|
|
|
|
|
|
bool ExecuteSaveFile(HWND owner,
|
|
|
|
+ const std::u16string& title,
|
|
|
|
const base::FilePath& default_path,
|
|
|
|
const std::vector<FileFilterSpec>& filter,
|
|
|
|
const std::wstring& def_ext,
|
2023-11-21 20:17:55 +01:00
|
|
|
@@ -394,9 +391,7 @@ bool ExecuteSaveFile(HWND owner,
|
2022-04-15 21:55:23 +02:00
|
|
|
|
|
|
|
DWORD dialog_options = FOS_OVERWRITEPROMPT;
|
|
|
|
|
|
|
|
- // Note: The title is not passed down for historical reasons.
|
|
|
|
- // TODO(pmonette): Figure out if it's a worthwhile improvement.
|
|
|
|
- return RunSaveFileDialog(owner, std::u16string(), default_path, filter,
|
|
|
|
+ return RunSaveFileDialog(owner, title, default_path, filter,
|
|
|
|
dialog_options, def_ext, filter_index, path);
|
|
|
|
}
|
|
|
|
|
2023-11-21 20:17:55 +01:00
|
|
|
@@ -421,7 +416,7 @@ void ExecuteSelectFile(
|
2022-04-15 21:55:23 +02:00
|
|
|
break;
|
|
|
|
case SelectFileDialog::SELECT_SAVEAS_FILE: {
|
|
|
|
base::FilePath path;
|
|
|
|
- if (ExecuteSaveFile(owner, default_path, filter, default_extension,
|
|
|
|
+ if (ExecuteSaveFile(owner, title, default_path, filter, default_extension,
|
|
|
|
&file_type_index, &path)) {
|
|
|
|
paths.push_back(std::move(path));
|
|
|
|
}
|
|
|
|
diff --git ui/shell_dialogs/select_file_dialog.cc ui/shell_dialogs/select_file_dialog.cc
|
2024-01-26 03:12:43 +01:00
|
|
|
index 68dd62159b686..e94831cd44a2d 100644
|
2022-04-15 21:55:23 +02:00
|
|
|
--- ui/shell_dialogs/select_file_dialog.cc
|
|
|
|
+++ ui/shell_dialogs/select_file_dialog.cc
|
2024-01-26 03:12:43 +01:00
|
|
|
@@ -73,8 +73,10 @@ void SelectFileDialog::SetFactory(
|
2022-04-15 21:55:23 +02:00
|
|
|
// static
|
|
|
|
scoped_refptr<SelectFileDialog> SelectFileDialog::Create(
|
|
|
|
Listener* listener,
|
|
|
|
- std::unique_ptr<ui::SelectFilePolicy> policy) {
|
|
|
|
- if (dialog_factory_)
|
|
|
|
+ std::unique_ptr<ui::SelectFilePolicy> policy,
|
|
|
|
+ bool run_from_cef) {
|
|
|
|
+ // Avoid reentrancy of the CEF factory.
|
|
|
|
+ if (dialog_factory_ && (!run_from_cef || !dialog_factory_->IsCefFactory()))
|
|
|
|
return dialog_factory_->Create(listener, std::move(policy));
|
|
|
|
return CreateSelectFileDialog(listener, std::move(policy));
|
|
|
|
}
|
|
|
|
diff --git ui/shell_dialogs/select_file_dialog.h ui/shell_dialogs/select_file_dialog.h
|
2024-05-29 22:38:00 +02:00
|
|
|
index 9b12fae59c3cc..a80873f421c8d 100644
|
2022-04-15 21:55:23 +02:00
|
|
|
--- ui/shell_dialogs/select_file_dialog.h
|
|
|
|
+++ ui/shell_dialogs/select_file_dialog.h
|
2024-01-26 03:12:43 +01:00
|
|
|
@@ -102,7 +102,8 @@ class SHELL_DIALOGS_EXPORT SelectFileDialog
|
2022-04-15 21:55:23 +02:00
|
|
|
// is refcounted and uses a background thread.
|
|
|
|
static scoped_refptr<SelectFileDialog> Create(
|
|
|
|
Listener* listener,
|
|
|
|
- std::unique_ptr<SelectFilePolicy> policy);
|
|
|
|
+ std::unique_ptr<SelectFilePolicy> policy,
|
|
|
|
+ bool run_from_cef = false);
|
|
|
|
|
|
|
|
SelectFileDialog(const SelectFileDialog&) = delete;
|
|
|
|
SelectFileDialog& operator=(const SelectFileDialog&) = delete;
|
2024-05-29 22:38:00 +02:00
|
|
|
@@ -126,6 +127,10 @@ class SHELL_DIALOGS_EXPORT SelectFileDialog
|
|
|
|
// be used.
|
|
|
|
std::vector<std::u16string> extension_description_overrides;
|
|
|
|
|
|
|
|
+ // Original mime types for the specified extensions. Entries correspond to
|
|
|
|
+ // |extensions|; if left blank then there was no mime type.
|
|
|
|
+ std::vector<std::u16string> extension_mimetypes;
|
|
|
|
+
|
|
|
|
// Specifies whether there will be a filter added for all files (i.e. *.*).
|
|
|
|
bool include_all_files = false;
|
|
|
|
|
|
|
|
@@ -199,6 +204,19 @@ class SHELL_DIALOGS_EXPORT SelectFileDialog
|
2022-10-17 19:27:40 +02:00
|
|
|
const GURL* caller = nullptr);
|
2022-04-15 21:55:23 +02:00
|
|
|
bool HasMultipleFileTypeChoices();
|
|
|
|
|
|
|
|
+ // Match the types used by CefWindowHandle.
|
|
|
|
+#if BUILDFLAG(IS_MAC)
|
|
|
|
+ using WidgetType = void*;
|
|
|
|
+ static constexpr WidgetType kNullWidget = nullptr;
|
|
|
|
+#else
|
|
|
|
+ using WidgetType = gfx::AcceleratedWidget;
|
|
|
|
+ static constexpr WidgetType kNullWidget = gfx::kNullAcceleratedWidget;
|
|
|
|
+#endif
|
|
|
|
+
|
|
|
|
+ void set_owning_widget(WidgetType widget) {
|
|
|
|
+ owning_widget_ = widget;
|
|
|
|
+ }
|
|
|
|
+
|
|
|
|
protected:
|
|
|
|
friend class base::RefCountedThreadSafe<SelectFileDialog>;
|
|
|
|
|
2024-05-29 22:38:00 +02:00
|
|
|
@@ -224,6 +242,11 @@ class SHELL_DIALOGS_EXPORT SelectFileDialog
|
2022-04-15 21:55:23 +02:00
|
|
|
// The listener to be notified of selection completion.
|
2023-09-15 21:51:43 +02:00
|
|
|
raw_ptr<Listener> listener_;
|
2022-04-15 21:55:23 +02:00
|
|
|
|
|
|
|
+ std::unique_ptr<SelectFilePolicy> select_file_policy_;
|
|
|
|
+
|
|
|
|
+ // Support override of the |owning_window| value.
|
|
|
|
+ WidgetType owning_widget_ = kNullWidget;
|
|
|
|
+
|
|
|
|
private:
|
|
|
|
// Tests if the file selection dialog can be displayed by
|
|
|
|
// testing if the AllowFileSelectionDialogs-Policy is
|
2024-05-29 22:38:00 +02:00
|
|
|
@@ -236,8 +259,6 @@ class SHELL_DIALOGS_EXPORT SelectFileDialog
|
2022-04-15 21:55:23 +02:00
|
|
|
|
|
|
|
// Returns true if the dialog has multiple file type choices.
|
|
|
|
virtual bool HasMultipleFileTypeChoicesImpl() = 0;
|
|
|
|
-
|
|
|
|
- std::unique_ptr<SelectFilePolicy> select_file_policy_;
|
|
|
|
};
|
|
|
|
|
|
|
|
SelectFileDialog* CreateSelectFileDialog(
|
|
|
|
diff --git ui/shell_dialogs/select_file_dialog_factory.h ui/shell_dialogs/select_file_dialog_factory.h
|
2022-10-17 19:27:40 +02:00
|
|
|
index c7acd9b05fbb8..3e95e4125fa24 100644
|
2022-04-15 21:55:23 +02:00
|
|
|
--- ui/shell_dialogs/select_file_dialog_factory.h
|
|
|
|
+++ ui/shell_dialogs/select_file_dialog_factory.h
|
|
|
|
@@ -24,6 +24,8 @@ class SHELL_DIALOGS_EXPORT SelectFileDialogFactory {
|
|
|
|
virtual SelectFileDialog* Create(
|
|
|
|
ui::SelectFileDialog::Listener* listener,
|
|
|
|
std::unique_ptr<ui::SelectFilePolicy> policy) = 0;
|
|
|
|
+
|
|
|
|
+ virtual bool IsCefFactory() const { return false; }
|
|
|
|
};
|
|
|
|
|
|
|
|
} // namespace ui
|
|
|
|
diff --git ui/shell_dialogs/select_file_dialog_mac.mm ui/shell_dialogs/select_file_dialog_mac.mm
|
2024-01-26 03:12:43 +01:00
|
|
|
index bf99047fa53cf..0efdb76233331 100644
|
2022-04-15 21:55:23 +02:00
|
|
|
--- ui/shell_dialogs/select_file_dialog_mac.mm
|
|
|
|
+++ ui/shell_dialogs/select_file_dialog_mac.mm
|
2024-01-26 03:12:43 +01:00
|
|
|
@@ -107,6 +107,10 @@ void SelectFileDialogImpl::SelectFileImpl(
|
2022-04-15 21:55:23 +02:00
|
|
|
mojo_window->CreateSelectFileDialog(std::move(receiver));
|
|
|
|
} else {
|
|
|
|
NSWindow* ns_window = gfx_window.GetNativeNSWindow();
|
|
|
|
+ if (!ns_window && owning_widget_) {
|
|
|
|
+ NSView* view = ((__bridge NSView*)owning_widget_);
|
|
|
|
+ ns_window = [view window];
|
|
|
|
+ }
|
|
|
|
mojo::MakeSelfOwnedReceiver(
|
|
|
|
std::make_unique<remote_cocoa::SelectFileDialogBridge>(ns_window),
|
|
|
|
std::move(receiver));
|
|
|
|
diff --git ui/shell_dialogs/select_file_dialog_win.cc ui/shell_dialogs/select_file_dialog_win.cc
|
2024-05-23 03:52:35 +02:00
|
|
|
index e5d4311ba152c..16c86e544fe3a 100644
|
2022-04-15 21:55:23 +02:00
|
|
|
--- ui/shell_dialogs/select_file_dialog_win.cc
|
|
|
|
+++ ui/shell_dialogs/select_file_dialog_win.cc
|
2024-05-23 03:52:35 +02:00
|
|
|
@@ -254,6 +254,8 @@ void SelectFileDialogImpl::SelectFileImpl(
|
2022-04-15 21:55:23 +02:00
|
|
|
HWND owner = owning_window && owning_window->GetRootWindow()
|
|
|
|
? owning_window->GetHost()->GetAcceleratedWidget()
|
|
|
|
: nullptr;
|
|
|
|
+ if (!owner)
|
|
|
|
+ owner = owning_widget_;
|
|
|
|
|
|
|
|
std::unique_ptr<RunState> run_state = BeginRun(owner);
|
|
|
|
|