DevTools: Inspect Element doesn't select element in the Elements Panel when tools are docked.

chrome/browser/debugger/devtools_manager.cc

 // Copyright (c) 2009 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/debugger/devtools_manager.h"
 
 #include <vector>
 
 #include "base/auto_reset.h"
 #include "base/message_loop.h"
@@ skipping 144 matching lines @@
   }
   if (enabled)
     it->second.insert(feature);
   else
     it->second.erase(feature);
 }
 
 void DevToolsManager::InspectElement(RenderViewHost* inspected_rvh,
                                      int x,
                                      int y) {
-  OpenDevToolsWindow(inspected_rvh);
   IPC::Message* m = new DevToolsAgentMsg_InspectElement(x, y);
   m->set_routing_id(inspected_rvh->routing_id());
   inspected_rvh->Send(m);
+  // TODO(loislo): we should initiate DevTools window opening from within
+  // renderer. Otherwise, we still can hit a race condition here.
+  OpenDevToolsWindow(inspected_rvh);
 }
 
 void DevToolsManager::ClientHostClosing(DevToolsClientHost* host) {
   RenderViewHost* inspected_rvh = GetInspectedRenderViewHost(host);
   if (!inspected_rvh) {
     // It might be in the list of orphan client hosts, remove it from there.
     for (OrphanClientHosts::iterator it = orphan_client_hosts_.begin();
          it != orphan_client_hosts_.end(); ++it) {
       if (it->second.first == host) {
         orphan_client_hosts_.erase(it->first);
@@ skipping 217 matching lines @@
                                        DevToolsClientHost* client_host) {
   DCHECK(inspected_rvh_to_client_host_.find(inspected_rvh)->second ==
       client_host);
   DCHECK(client_host_to_inspected_rvh_.find(client_host)->second ==
       inspected_rvh);
 
   inspected_rvh_to_client_host_.erase(inspected_rvh);
   client_host_to_inspected_rvh_.erase(client_host);
   runtime_features_map_.erase(inspected_rvh);
 }