From a878900d1577d0943387f16774a09a5ceae24b2f Mon Sep 17 00:00:00 2001
From: "pfeldman@chromium.org"
 <pfeldman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>
Date: Thu, 6 May 2010 09:56:58 +0000
Subject: DevTools: inspector tabs are initially drawn in the undocked style
 even for docked inspectors BUG=40804

Review URL: http://codereview.chromium.org/2009002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46555 0039d316-1c4b-4281-b951-d872f2087c98
---
 chrome/browser/debugger/devtools_window.cc | 23 +++++++++++++++++++----
 chrome/browser/debugger/devtools_window.h  |  1 +
 2 files changed, 20 insertions(+), 4 deletions(-)
 mode change 100644 => 100755 chrome/browser/debugger/devtools_window.h

diff --git a/chrome/browser/debugger/devtools_window.cc b/chrome/browser/debugger/devtools_window.cc
index debf10a..c8dbfd8 100755
--- a/chrome/browser/debugger/devtools_window.cc
+++ b/chrome/browser/debugger/devtools_window.cc
@@ -60,9 +60,8 @@ DevToolsWindow::DevToolsWindow(Profile* profile,
       open_console_on_load_(false) {
   // Create TabContents with devtools.
   tab_contents_ = new TabContents(profile, NULL, MSG_ROUTING_NONE, NULL);
-  GURL url(std::string(chrome::kChromeUIDevToolsURL) + "devtools.html");
   tab_contents_->render_view_host()->AllowBindings(BindingsPolicy::DOM_UI);
-  tab_contents_->controller().LoadURL(url, GURL(), PageTransition::START_PAGE);
+  tab_contents_->controller().LoadURL(GetDevToolsUrl(), GURL(), PageTransition::START_PAGE);
 
   // Wipe out page icon so that the default application icon is used.
   NavigationEntry* entry = tab_contents_->controller().GetActiveEntry();
@@ -241,9 +240,7 @@ void DevToolsWindow::Observe(NotificationType type,
                              const NotificationSource& source,
                              const NotificationDetails& details) {
   if (type == NotificationType::LOAD_STOP) {
-    SetAttachedWindow();
     is_loaded_ = true;
-    UpdateTheme();
     if (open_console_on_load_) {
       DoOpenConsole();
       open_console_on_load_ = false;
@@ -283,6 +280,24 @@ std::string SkColorToRGBAString(SkColor color) {
       DoubleToString(SkColorGetA(color) / 255.0).c_str());
 }
 
+GURL DevToolsWindow::GetDevToolsUrl() {
+  BrowserThemeProvider* tp = profile_->GetThemeProvider();
+  CHECK(tp);
+
+  SkColor color_toolbar =
+      tp->GetColor(BrowserThemeProvider::COLOR_TOOLBAR);
+  SkColor color_tab_text =
+      tp->GetColor(BrowserThemeProvider::COLOR_BOOKMARK_TEXT);
+
+  std::string url_string = StringPrintf(
+      "%sdevtools.html?docked=%s&toolbar_color=%s&text_color=%s",
+      chrome::kChromeUIDevToolsURL,
+      docked_ ? "true" : "false",
+      SkColorToRGBAString(color_toolbar).c_str(),
+      SkColorToRGBAString(color_tab_text).c_str());
+  return GURL(url_string);
+}
+
 void DevToolsWindow::UpdateTheme() {
   BrowserThemeProvider* tp = profile_->GetThemeProvider();
   CHECK(tp);
diff --git a/chrome/browser/debugger/devtools_window.h b/chrome/browser/debugger/devtools_window.h
old mode 100644
new mode 100755
index f799639..fbd14dc
--- a/chrome/browser/debugger/devtools_window.h
+++ b/chrome/browser/debugger/devtools_window.h
@@ -61,6 +61,7 @@ class DevToolsWindow
 
   void ScheduleOpenConsole();
   void DoOpenConsole();
+  GURL GetDevToolsUrl();
   void UpdateTheme();
 
   // Overridden from TabContentsDelegate.
-- 
cgit v1.1