From 35c90fd807a140101e60020c077388688cf777f4 Mon Sep 17 00:00:00 2001
From: Charles Kerr <charles@charleskerr.com>
Date: Thu, 27 Mar 2025 14:32:01 -0500
Subject: [PATCH] perf: avoid double map lookup in
 `WebFrameMain::UpdateRenderFrameHost()` (#46293)

perf: avoid double map lookup in WebFrameMain::UpdateRenderFrameHost()
---
 shell/browser/api/electron_api_web_frame_main.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/shell/browser/api/electron_api_web_frame_main.cc b/shell/browser/api/electron_api_web_frame_main.cc
index ce35a402c527..1bf82842a04f 100644
--- a/shell/browser/api/electron_api_web_frame_main.cc
+++ b/shell/browser/api/electron_api_web_frame_main.cc
@@ -193,8 +193,8 @@ void WebFrameMain::UpdateRenderFrameHost(content::RenderFrameHost* rfh) {
   // Ensure that RFH being swapped in doesn't already exist as its own
   // WebFrameMain instance.
   frame_token_ = rfh->GetGlobalFrameToken();
-  DCHECK(!GetFrameTokenMap().contains(frame_token_));
-  GetFrameTokenMap().emplace(frame_token_, this);
+  const auto [_, inserted] = GetFrameTokenMap().emplace(frame_token_, this);
+  DCHECK(inserted);
 
   render_frame_disposed_ = false;
   TeardownMojoConnection();