From 8d58999135995bdd3032062c01623dee08d38ac2 Mon Sep 17 00:00:00 2001 From: Charles Kerr Date: Fri, 21 Feb 2025 15:28:00 -0600 Subject: [PATCH] refactor: use base's map utills in WebFrameMain (#45741) refactor: use base::FindPtrOrNull() in WebFrameMain::FromFrameTreeNodeId() refactor: use base::FindPtrOrNull() in WebFrameMain::FromFrameToken() --- shell/browser/api/electron_api_web_frame_main.cc | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/shell/browser/api/electron_api_web_frame_main.cc b/shell/browser/api/electron_api_web_frame_main.cc index 8c4b7778f222..4936dc7806cc 100644 --- a/shell/browser/api/electron_api_web_frame_main.cc +++ b/shell/browser/api/electron_api_web_frame_main.cc @@ -9,6 +9,7 @@ #include #include +#include "base/containers/map_util.h" #include "base/feature_list.h" #include "base/logging.h" #include "base/no_destructor.h" @@ -121,19 +122,13 @@ FrameTokenMap& GetFrameTokenMap() { // static WebFrameMain* WebFrameMain::FromFrameTreeNodeId( content::FrameTreeNodeId frame_tree_node_id) { - FrameTreeNodeIdMap& frame_map = GetFrameTreeNodeIdMap(); - auto iter = frame_map.find(frame_tree_node_id); - auto* web_frame = iter == frame_map.end() ? nullptr : iter->second; - return web_frame; + return base::FindPtrOrNull(GetFrameTreeNodeIdMap(), frame_tree_node_id); } // static WebFrameMain* WebFrameMain::FromFrameToken( content::GlobalRenderFrameHostToken frame_token) { - FrameTokenMap& frame_map = GetFrameTokenMap(); - auto iter = frame_map.find(frame_token); - auto* web_frame = iter == frame_map.end() ? nullptr : iter->second; - return web_frame; + return base::FindPtrOrNull(GetFrameTokenMap(), frame_token); } // static