Title: [101192] trunk/Source/WebCore
Revision
101192
Author
[email protected]
Date
2011-11-26 04:48:24 -0800 (Sat, 26 Nov 2011)

Log Message

Fix a bug in TreeOutline which would cause parent elements to show up as expanded and empty
when removed and added back to the tree.

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

Reviewed by Pavel Feldman.

* inspector/front-end/treeoutline.js:
(TreeElement.prototype.collapse): Set _treeElementsExpandedState to false instead of true.
Only TreeElement.prototype.expand should set it to true, which it does.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (101191 => 101192)


--- trunk/Source/WebCore/ChangeLog	2011-11-26 10:47:48 UTC (rev 101191)
+++ trunk/Source/WebCore/ChangeLog	2011-11-26 12:48:24 UTC (rev 101192)
@@ -1,3 +1,16 @@
+2011-11-26  Timothy Hatcher  <[email protected]>
+
+        Fix a bug in TreeOutline which would cause parent elements to show up as expanded and empty
+        when removed and added back to the tree.
+
+        https://bugs.webkit.org/show_bug.cgi?id=73155
+
+        Reviewed by Pavel Feldman.
+
+        * inspector/front-end/treeoutline.js:
+        (TreeElement.prototype.collapse): Set _treeElementsExpandedState to false instead of true.
+        Only TreeElement.prototype.expand should set it to true, which it does.
+
 2011-11-26  Martin Robinson  <[email protected]>
 
         [GTK] Fix some warnings in WebCore GTK+

Modified: trunk/Source/WebCore/inspector/front-end/treeoutline.js (101191 => 101192)


--- trunk/Source/WebCore/inspector/front-end/treeoutline.js	2011-11-26 10:47:48 UTC (rev 101191)
+++ trunk/Source/WebCore/inspector/front-end/treeoutline.js	2011-11-26 12:48:24 UTC (rev 101192)
@@ -706,7 +706,7 @@
 
     this.expanded = false;
     if (this.treeOutline)
-        this.treeOutline._treeElementsExpandedState[this.identifier] = true;
+        this.treeOutline._treeElementsExpandedState[this.identifier] = false;
 
     if (this.oncollapse)
         this.oncollapse(this);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to