[DevTools] Move InspectElement call after OpenDevToolsWindow

content/renderer/devtools/devtools_agent.cc

 // Copyright (c) 2012 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 "content/renderer/devtools/devtools_agent.h"
 
 #include <map>
 
 #include "base/lazy_instance.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 201 matching lines @@
 void DevToolsAgent::OnDispatchOnInspectorBackend(const std::string& message) {
   TRACE_EVENT0("devtools", "DevToolsAgent::OnDispatchOnInspectorBackend");
   WebDevToolsAgent* web_agent = GetWebAgent();
   if (web_agent)
     web_agent->dispatchOnInspectorBackend(WebString::fromUTF8(message));
 }
 
 void DevToolsAgent::OnInspectElement(
     const std::string& host_id, int x, int y) {
   WebDevToolsAgent* web_agent = GetWebAgent();
   if (web_agent) {

[dgozman, 2015/11/13 19:21:57]

Can you check that we are attached already?

[kozy, 2015/11/15 00:39:05]

Done.

-    web_agent->attach(WebString::fromUTF8(host_id));
     web_agent->inspectElementAt(WebPoint(x, y));
     is_attached_ = true;

[dgozman, 2015/11/13 19:21:57]

Remove this.

[kozy, 2015/11/15 00:39:05]

Done.

   }
 }
 
 void DevToolsAgent::AddMessageToConsole(ConsoleMessageLevel level,
                                         const std::string& message) {
   WebLocalFrame* web_frame = frame_->GetWebFrame();
   if (!web_frame)
     return;
 
   WebConsoleMessage::Level target_level = WebConsoleMessage::LevelLog;
@@ skipping 34 matching lines @@
 WebDevToolsAgent* DevToolsAgent::GetWebAgent() {
   WebLocalFrame* web_frame = frame_->GetWebFrame();
   return web_frame ? web_frame->devToolsAgent() : nullptr;
 }
 
 bool DevToolsAgent::IsAttached() {
   return is_attached_;
 }
 
 }  // namespace content