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

perf: avoid std::map temporaries in WebContents::DevToolsRequestFileSystems() (#46309)

Browse source 
* perf: move the GetDevToolsWebContents() call outside of the loop

Co-authored-by: Charles Kerr <charles@charleskerr.com>

* perf: remove std::map temporary in WebContents::DevToolsRequestFileSystems()

Co-authored-by: Charles Kerr <charles@charleskerr.com>

* refactor: remove unused GetAddedFileSystemPaths()

Co-authored-by: Charles Kerr <charles@charleskerr.com>

* perf: remove std::vector temporary in WebContents::DevToolsRequestFileSystems()

Co-authored-by: Charles Kerr <charles@charleskerr.com>

---------

Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com>
Co-authored-by: Charles Kerr <charles@charleskerr.com>
This commit is contained in:
trop[bot] 2025-03-26 19:25:49 -05:00 • committed by GitHub
parent 3ff5c4ea5f
commit dde045db9c
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 13 additions and 33 deletions
Download patch file Download diff file Expand all files Collapse all files

46
shell/browser/api/electron_api_web_contents.cc
View file

@ -682,17 +682,6 @@ PrefService* GetPrefService(content::WebContents* web_contents) {
return GetPrefService(web_contents)->GetDict(prefs::kDevToolsFileSystemPaths);
}
std::map<std::string, std::string> GetAddedFileSystemPaths(
content::WebContents* web_contents) {
std::map<std::string, std::string> result;
for (auto it : GetAddedFileSystems(web_contents)) {
std::string type =
it.second.is_string() ? it.second.GetString() : std::string();
result[it.first] = type;
}
return result;
}
bool IsDevToolsFileSystemAdded(content::WebContents* web_contents,
const std::string_view file_system_path) {
return GetAddedFileSystems(web_contents).contains(file_system_path);
@ -4053,31 +4042,22 @@ void WebContents::DevToolsAppendToFile(const std::string& url,
}
void WebContents::DevToolsRequestFileSystems() {
auto file_system_paths = GetAddedFileSystemPaths(GetDevToolsWebContents());
if (file_system_paths.empty()) {
inspectable_web_contents_->CallClientFunction(
"DevToolsAPI", "fileSystemsLoaded", base::Value(base::Value::List()));
return;
const std::string empty_str;
content::WebContents* const dtwc = GetDevToolsWebContents();
const base::Value::Dict& added_paths = GetAddedFileSystems(dtwc);
auto filesystems = base::Value::List::with_capacity(added_paths.size());
for (const auto path_and_type : added_paths) {
const auto& [path, type_val] = path_and_type;
const auto& type = type_val.is_string() ? type_val.GetString() : empty_str;
const std::string file_system_id =
RegisterFileSystem(dtwc, base::FilePath::FromUTF8Unsafe(path));
filesystems.Append(CreateFileSystemValue(
CreateFileSystemStruct(dtwc, file_system_id, path, type)));
}
std::vector<FileSystem> file_systems;
for (const auto& file_system_path : file_system_paths) {
base::FilePath path =
base::FilePath::FromUTF8Unsafe(file_system_path.first);
std::string file_system_id =
RegisterFileSystem(GetDevToolsWebContents(), path);
FileSystem file_system =
CreateFileSystemStruct(GetDevToolsWebContents(), file_system_id,
file_system_path.first, file_system_path.second);
file_systems.push_back(file_system);
}
base::Value::List file_system_value;
for (const auto& file_system : file_systems)
file_system_value.Append(CreateFileSystemValue(file_system));
inspectable_web_contents_->CallClientFunction(
"DevToolsAPI", "fileSystemsLoaded",
base::Value(std::move(file_system_value)));
"DevToolsAPI", "fileSystemsLoaded", base::Value{std::move(filesystems)});
}
void WebContents::DevToolsAddFileSystem(