mirrors/electron
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions 1

fix: getAsFileSystemHandle failure when drag-dropping two directories (#45234)

Browse source 
fix: drag-dropping two directories
This commit is contained in:
Shelley Vohr 2025-01-20 09:54:12 +01:00 • committed by GitHub
parent 6953f5505f
commit 0e5fe3fa60
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 17 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

19
shell/browser/file_system_access/file_system_access_permission_context.cc
View file

@ -588,7 +588,7 @@ void FileSystemAccessPermissionContext::ConfirmSensitiveEntryAccess(
content::GlobalRenderFrameHostId frame_id,
base::OnceCallback<void(SensitiveEntryResult)> callback) {
DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
callback_ = std::move(callback);
callback_map_[path_info.path] = std::move(callback);
auto after_blocklist_check_callback = base::BindOnce(
&FileSystemAccessPermissionContext::DidCheckPathAgainstBlocklist,
@ -632,16 +632,18 @@ void FileSystemAccessPermissionContext::PerformAfterWriteChecks(
}
void FileSystemAccessPermissionContext::RunRestrictedPathCallback(
const base::FilePath& file_path,
SensitiveEntryResult result) {
if (callback_)
std::move(callback_).Run(result);
if (base::Contains(callback_map_, file_path))
std::move(callback_map_[file_path]).Run(result);
}
void FileSystemAccessPermissionContext::OnRestrictedPathResult(
const base::FilePath& file_path,
gin::Arguments* args) {
SensitiveEntryResult result = SensitiveEntryResult::kAbort;
args->GetNext(&result);
RunRestrictedPathCallback(result);
RunRestrictedPathCallback(file_path, result);
}
void FileSystemAccessPermissionContext::DidCheckPathAgainstBlocklist(
@ -654,8 +656,9 @@ void FileSystemAccessPermissionContext::DidCheckPathAgainstBlocklist(
DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
if (user_action == UserAction::kNone) {
RunRestrictedPathCallback(should_block ? SensitiveEntryResult::kAbort
: SensitiveEntryResult::kAllowed);
auto result = should_block ? SensitiveEntryResult::kAbort
: SensitiveEntryResult::kAllowed;
RunRestrictedPathCallback(path_info.path, result);
return;
}
@ -674,11 +677,11 @@ void FileSystemAccessPermissionContext::DidCheckPathAgainstBlocklist(
"file-system-access-restricted", details,
base::BindRepeating(
&FileSystemAccessPermissionContext::OnRestrictedPathResult,
weak_factory_.GetWeakPtr()));
weak_factory_.GetWeakPtr(), path_info.path));
return;
}
RunRestrictedPathCallback(SensitiveEntryResult::kAllowed);
RunRestrictedPathCallback(path_info.path, SensitiveEntryResult::kAllowed);
}
void FileSystemAccessPermissionContext::MaybeEvictEntries(

9
shell/browser/file_system_access/file_system_access_permission_context.h
View file

@ -144,9 +144,11 @@ class FileSystemAccessPermissionContext
content::GlobalRenderFrameHostId frame_id,
bool should_block);
void RunRestrictedPathCallback(SensitiveEntryResult result);
void RunRestrictedPathCallback(const base::FilePath& file_path,
SensitiveEntryResult result);
void OnRestrictedPathResult(gin::Arguments* args);
void OnRestrictedPathResult(const base::FilePath& file_path,
gin::Arguments* args);
void MaybeEvictEntries(base::Value::Dict& dict);
@ -170,7 +172,8 @@ class FileSystemAccessPermissionContext
std::map<url::Origin, base::Value::Dict> id_pathinfo_map_;
base::OnceCallback<void(SensitiveEntryResult)> callback_;
std::map<base::FilePath, base::OnceCallback<void(SensitiveEntryResult)>>
callback_map_;
base::WeakPtrFactory<FileSystemAccessPermissionContext> weak_factory_{this};
};