Web Inspector: inspector close button is missing in the dock-to-right mode.

https://bugs.webkit.org/show_bug.cgi?id=76829

Reviewed by Timothy Hatcher.

* inspector/front-end/inspector.js:
(WebInspector.set attached):
(WebInspector.get _setCompactMode):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@105623 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
index e1bdec3..aad4750 100644 (file)
--- a/Source/WebCore/ChangeLog
+++ b/Source/WebCore/ChangeLog
@@ -1,3 +1,14 @@
+2012-01-23  Pavel Feldman  <pfeldman@google.com>
+
+        Web Inspector: inspector close button is missing in the dock-to-right mode.
+        https://bugs.webkit.org/show_bug.cgi?id=76829
+
+        Reviewed by Timothy Hatcher.
+
+        * inspector/front-end/inspector.js:
+        (WebInspector.set attached):
+        (WebInspector.get _setCompactMode):
+
 2012-01-23  Vsevolod Vlasov  <vsevik@chromium.org>
 
         Web Inspector: IndexedDBModel should keep track of requests sent to the backend.

diff --git a/Source/WebCore/inspector/front-end/inspector.js b/Source/WebCore/inspector/front-end/inspector.js
index 20675bc..c75033e 100644 (file)
--- a/Source/WebCore/inspector/front-end/inspector.js
+++ b/Source/WebCore/inspector/front-end/inspector.js
@@ -187,6 +187,11 @@ var WebInspector = {
             this._dockToggleButton.toggled = !x;
         }
 
+        if (x)
+            document.body.removeStyleClass("detached");
+        else
+            document.body.addStyleClass("detached");
+
         this._setCompactMode(x && !WebInspector.settings.dockToRight.get());
     },
 
@@ -198,13 +203,10 @@ var WebInspector = {
     _setCompactMode: function(x)
     {
         var body = document.body;
-        if (x) {
-            body.removeStyleClass("detached");
+        if (x)
             body.addStyleClass("compact");
-        } else {
+        else
             body.removeStyleClass("compact");
-            body.addStyleClass("detached");
-        }
 
         // This may be called before doLoadedDone, hence the bulk of inspector objects may
         // not be created yet.