DevTools: check for target being oopif prior to matching its web contents.

chrome/browser/devtools/chrome_devtools_manager_delegate.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/devtools/chrome_devtools_manager_delegate.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "build/build_config.h"
 #include "chrome/browser/devtools/devtools_network_protocol_handler.h"
 #include "chrome/browser/devtools/devtools_window.h"
@@ skipping 52 matching lines @@
   content::WebContents* web_contents =
       content::WebContents::FromRenderFrameHost(host);
 
   guest_view::GuestViewBase* guest =
       guest_view::GuestViewBase::FromWebContents(web_contents);
   content::WebContents* guest_contents =
       guest ? guest->embedder_web_contents() : nullptr;
   if (guest_contents)
     return kTypeWebView;
 
+  if (host->GetParent())
+    return content::DevToolsAgentHost::kTypeFrame;
+
   for (TabContentsIterator it; !it.done(); it.Next()) {
     if (*it == web_contents)
       return content::DevToolsAgentHost::kTypePage;
   }
 
-  if (host->GetParent())
-    return content::DevToolsAgentHost::kTypeFrame;
-
   const extensions::Extension* extension = extensions::ExtensionRegistry::Get(
       web_contents->GetBrowserContext())->enabled_extensions().GetByID(
           host->GetLastCommittedURL().host());
   if (!extension)
     return content::DevToolsAgentHost::kTypeOther;
 
   Profile* profile =
       Profile::FromBrowserContext(web_contents->GetBrowserContext());
   if (!profile)
     return content::DevToolsAgentHost::kTypeOther;
@@ skipping 39 matching lines @@
   if (!params.target_contents)
     return nullptr;
   return content::DevToolsAgentHost::GetOrCreateFor(params.target_contents);
 }
 
 void ChromeDevToolsManagerDelegate::DevToolsAgentStateChanged(
     content::DevToolsAgentHost* agent_host,
     bool attached) {
   network_protocol_handler_->DevToolsAgentStateChanged(agent_host, attached);
 }