DevTools: devtools manager should know nothing about DevToolsWindow

content/browser/debugger/devtools_window.cc

 // Copyright (c) 2011 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 <algorithm>
+
 #include "base/command_line.h"
 #include "base/json/json_writer.h"
 #include "base/stringprintf.h"
 #include "base/string_number_conversions.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/prefs/scoped_user_pref_update.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/sessions/restore_tab_helper.h"
 #include "chrome/browser/tabs/tab_strip_model.h"
 #include "chrome/browser/themes/theme_service.h"
 #include "chrome/browser/themes/theme_service_factory.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/render_messages.h"
 #include "chrome/common/url_constants.h"
+#include "content/browser/browsing_instance.h"
 #include "content/browser/debugger/devtools_manager.h"
 #include "content/browser/debugger/devtools_window.h"
 #include "content/browser/in_process_webkit/session_storage_namespace.h"
 #include "content/browser/load_notification_details.h"
 #include "content/browser/renderer_host/render_view_host.h"
 #include "content/browser/tab_contents/navigation_controller.h"
 #include "content/browser/tab_contents/navigation_entry.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "content/browser/tab_contents/tab_contents_view.h"
 #include "content/common/bindings_policy.h"
 #include "content/common/notification_service.h"
 #include "grit/generated_resources.h"
 
 const char DevToolsWindow::kDevToolsApp[] = "DevToolsApp";
 
+DevToolsWindow::DevToolsWindowList DevToolsWindow::instances_;
+
 // static
 TabContentsWrapper* DevToolsWindow::GetDevToolsContents(
     TabContents* inspected_tab) {
   if (!inspected_tab) {
     return NULL;
   }
 
-  if (!DevToolsManager::GetInstance())
+  DevToolsManager* manager = DevToolsManager::GetInstance();
+  if (!manager)
     return NULL;  // Happens only in tests.
 
-  DevToolsClientHost* client_host = DevToolsManager::GetInstance()->
-          GetDevToolsClientHostFor(inspected_tab->render_view_host());
-  if (!client_host) {
-    return NULL;
-  }
-
-  DevToolsWindow* window = client_host->AsDevToolsWindow();
+  DevToolsClientHost* client_host = manager->
+      GetDevToolsClientHostFor(inspected_tab->render_view_host());
+  DevToolsWindow* window = AsDevToolsWindow(client_host);
   if (!window || !window->is_docked()) {
     return NULL;
   }
   return window->tab_contents();
 }
 
+// static
+DevToolsWindow* DevToolsWindow::FindDevToolsWindow(
+    RenderViewHost* window_rvh) {
+  for (DevToolsWindowList::iterator it = instances_.begin();
+       it != instances_.end(); ++it) {
+    if ((*it)->GetRenderViewHost() == window_rvh)
+      return *it;
+  }
+  return NULL;
+}
+
+// static
+void DevToolsWindow::OpenDevToolsWindow(RenderViewHost* inspected_rvh) {
+  ToggleDevToolsWindow(inspected_rvh, true, DEVTOOLS_TOGGLE_ACTION_NONE);
+}
+
+// static
+void DevToolsWindow::ToggleDevToolsWindow(RenderViewHost* inspected_rvh,
+                                          DevToolsToggleAction action) {
+  ToggleDevToolsWindow(inspected_rvh, false, action);
+}
+
+void DevToolsWindow::InspectElement(RenderViewHost* inspected_rvh,
+                                    int x,
+                                    int y) {
+  DevToolsManager::GetInstance()->SendInspectElement(inspected_rvh, x, y);
+  // TODO(loislo): we should initiate DevTools window opening from within
+  // renderer. Otherwise, we still can hit a race condition here.
+  OpenDevToolsWindow(inspected_rvh);
+}
+
+
 DevToolsWindow::DevToolsWindow(Profile* profile,
                                RenderViewHost* inspected_rvh,
                                bool docked)
     : profile_(profile),
       inspected_tab_(NULL),
       browser_(NULL),
       docked_(docked),
       is_loaded_(false),
       action_on_load_(DEVTOOLS_TOGGLE_ACTION_NONE) {
   // Create TabContents with devtools.
@@ skipping 16 matching lines @@
   registrar_.Add(this,
                  NotificationType::TAB_CLOSING,
                  Source<NavigationController>(&tab_contents_->controller()));
   registrar_.Add(
       this,
       NotificationType::BROWSER_THEME_CHANGED,
       Source<ThemeService>(ThemeServiceFactory::GetForProfile(profile_)));
   TabContents* tab = inspected_rvh->delegate()->GetAsTabContents();
   if (tab)
     inspected_tab_ = TabContentsWrapper::GetCurrentWrapperForContents(tab);
+
+  instances_.push_back(this);
 }
 
 DevToolsWindow::~DevToolsWindow() {
-}
-
-DevToolsWindow* DevToolsWindow::AsDevToolsWindow() {
-  return this;
+  DevToolsWindowList::iterator it = std::find(instances_.begin(),
+                                              instances_.end(),
+                                              this);
+  DCHECK(it != instances_.end());
+  instances_.erase(it);
 }
 
 void DevToolsWindow::SendMessageToClient(const IPC::Message& message) {
   RenderViewHost* target_host =
       tab_contents_->tab_contents()->render_view_host();
   IPC::Message* m =  new IPC::Message(message);
   m->set_routing_id(target_host->routing_id());
   target_host->Send(m);
 }
 
@@ skipping 71 matching lines @@
   } else {
     BrowserWindow* inspected_window = GetInspectedBrowserWindow();
     if (inspected_window)
       tab_contents_->view()->Focus();
   }
 }
 
 void DevToolsWindow::SetDocked(bool docked) {
   if (docked_ == docked)
     return;
+
+  if (!inspected_tab_)
+    return;
+  RenderViewHost* inspected_rvh = inspected_tab_->render_view_host();

[pfeldman, 2011/07/07 12:17:52]

Do you need this code?

[yurys, 2011/07/07 12:30:40]

Removed.

+  DCHECK(inspected_rvh);
+  profile_->GetPrefs()->SetBoolean(prefs::kDevToolsOpenDocked, docked);
+
   if (docked && (!GetInspectedBrowserWindow() ||
                  IsInspectedBrowserPopupOrPanel())) {
     // Cannot dock, avoid window flashing due to close-reopen cycle.
     return;
   }
   docked_ = docked;
 
   if (docked) {
     // Detach window from the external devtools browser. It will lead to
     // the browser object's close and delete. Remove observer first.
@@ skipping 262 matching lines @@
   if (docked_) {
     if (event.windowsKeyCode == 0x08) {
       // Do not navigate back in history on Windows (http://crbug.com/74156).
       return;
     }
     BrowserWindow* inspected_window = GetInspectedBrowserWindow();
     if (inspected_window)
       inspected_window->HandleKeyboardEvent(event);
   }
 }
+
+// static
+void DevToolsWindow::ToggleDevToolsWindow(
+    RenderViewHost* inspected_rvh,
+    bool force_open,
+    DevToolsToggleAction action) {
+  DevToolsManager* manager = DevToolsManager::GetInstance();
+  DevToolsWindow* window = NULL;
+
+  DevToolsClientHost* host = manager->GetDevToolsClientHostFor(inspected_rvh);
+  if (host != NULL && AsDevToolsWindow(host) == NULL) {
+    // Break remote debugging / extension debugging session.
+    manager->UnregisterDevToolsClientHostFor(inspected_rvh);
+  }
+
+  bool do_open = force_open;
+  if (!window) {

[pfeldman, 2011/07/07 12:17:52]

if (!host)

[yurys, 2011/07/07 12:30:40]

The code was refactored.

+    Profile* profile = inspected_rvh->process()->profile();
+    bool docked = profile->GetPrefs()->GetBoolean(prefs::kDevToolsOpenDocked);
+    window = new DevToolsWindow(profile, inspected_rvh, docked);
+    manager->RegisterDevToolsClientHostFor(inspected_rvh, window);
+    do_open = true;
+  }

[pfeldman, 2011/07/07 12:17:52]

else {
  window = AsDevToolsWindow(host)
}

[yurys, 2011/07/07 12:30:40]

Rewrote this part.

+
+  // If window is docked and visible, we hide it on toggle. If window is
+  // undocked, we show (activate) it.
+  if (!window->is_docked() || do_open) {

[pfeldman, 2011/07/07 12:17:52]

remove {}

[yurys, 2011/07/07 12:30:40]

Done.

+    window->Show(action);
+  } else {
+    manager->UnregisterDevToolsClientHostFor(inspected_rvh);
+  }
+}
+
+// static
+DevToolsWindow* DevToolsWindow::AsDevToolsWindow(
+    DevToolsClientHost* client_host) {
+  if (!client_host)
+    return NULL;
+  DevToolsWindowList::iterator it = std::find(instances_.begin(),
+                                              instances_.end(),
+                                              client_host);
+  if (it == instances_.end()) {

[pfeldman, 2011/07/07 12:17:52]

ditto

[yurys, 2011/07/07 12:30:40]

Done.

+    return NULL;
+  }
+  return *it;
+}