Better fix for a redundant call to Reattach in RenderViewDevToolsAgentHost::ConnectRenderViewHost

content/browser/devtools/render_view_devtools_agent_host.cc

Index: content/browser/devtools/render_view_devtools_agent_host.cc
diff --git a/content/browser/devtools/render_view_devtools_agent_host.cc b/content/browser/devtools/render_view_devtools_agent_host.cc
index efc5242cf3bb1f763b54b57e599586bb507271f6..7455c160d8e5c89f70bdb803cd44ea597363315c 100644
--- a/content/browser/devtools/render_view_devtools_agent_host.cc
+++ b/content/browser/devtools/render_view_devtools_agent_host.cc
@@ -229,7 +229,7 @@ RenderViewDevToolsAgentHost::~RenderViewDevToolsAgentHost() {
 
 void RenderViewDevToolsAgentHost::AboutToNavigateRenderView(
     RenderViewHost* dest_rvh) {
-  if (!render_view_host_ || !IsAttached())
+  if (!render_view_host_)
     return;
 
   if (render_view_host_ == dest_rvh && static_cast<RenderViewHostImpl*>(
@@ -273,7 +273,8 @@ void RenderViewDevToolsAgentHost::SetRenderViewHost(RenderViewHost* rvh) {
 
 void RenderViewDevToolsAgentHost::ConnectRenderViewHost(RenderViewHost* rvh) {
   SetRenderViewHost(rvh);
-  Reattach(state_);
+  if (IsAttached())
+    Reattach(state_);
 }
 
 void RenderViewDevToolsAgentHost::DisconnectRenderViewHost() {