Fixes a bug where the Web Insepctor would jump back to
authortimothy@apple.com <timothy@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 23 May 2008 17:15:08 +0000 (17:15 +0000)
committertimothy@apple.com <timothy@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 23 May 2008 17:15:08 +0000 (17:15 +0000)
Elements each time it is opened instead of staying at the
last selected panel.

<rdar://problem/5959475> Inspector jumps to the Elements panel
each time it is opened, instead of staying at the previous panel

Reviewed by Adam Roben.

* page/InspectorController.cpp:
(WebCore::InspectorController::setWindowVisible): Reset
m_showAfterVisible to CurrentPanel, so the next time setWindowVisible
is called it will use CurrentPanel unless showPanel was called again.

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

WebCore/ChangeLog
WebCore/page/InspectorController.cpp

index 8b067c5..62a81d6 100644 (file)
@@ -1,3 +1,19 @@
+2008-05-23  Timothy Hatcher  <timothy@apple.com>
+
+        Fixes a bug where the Web Insepctor would jump back to
+        Elements each time it is opened instead of staying at the
+        last selected panel.
+
+        <rdar://problem/5959475> Inspector jumps to the Elements panel
+        each time it is opened, instead of staying at the previous panel
+
+        Reviewed by Adam Roben.
+
+        * page/InspectorController.cpp:
+        (WebCore::InspectorController::setWindowVisible): Reset
+        m_showAfterVisible to CurrentPanel, so the next time setWindowVisible
+        is called it will use CurrentPanel unless showPanel was called again.
+
 2008-05-23  chris fleizach  <cfleizach@apple.com>
 
         Reviewed by Darin Adler.
index 955e7f3..4e68781 100644 (file)
@@ -1176,6 +1176,8 @@ void InspectorController::setWindowVisible(bool visible)
             showPanel(m_showAfterVisible);
     } else
         resetScriptObjects();
+
+    m_showAfterVisible = CurrentPanel;
 }
 
 void InspectorController::addMessageToConsole(MessageSource source, MessageLevel level, ExecState* exec, const List& arguments, unsigned lineNumber, const String& sourceURL)