Title: [201843] trunk/Source/WebInspectorUI
- Revision
- 201843
- Author
- bb...@apple.com
- Date
- 2016-06-08 16:17:11 -0700 (Wed, 08 Jun 2016)
Log Message
Web Inspector: an error is encountered when trying to delete nodes that have children
https://bugs.webkit.org/show_bug.cgi?id=158541
<rdar://problem/26705901>
Reviewed by Timothy Hatcher.
* UserInterface/Views/DOMTreeElement.js:
(WebInspector.DOMTreeElement.prototype.adjustCollapsedRange):
Offset into the array variable instead of a variable holding its length.
Modified Paths
Diff
Modified: trunk/Source/WebInspectorUI/ChangeLog (201842 => 201843)
--- trunk/Source/WebInspectorUI/ChangeLog 2016-06-08 23:11:23 UTC (rev 201842)
+++ trunk/Source/WebInspectorUI/ChangeLog 2016-06-08 23:17:11 UTC (rev 201843)
@@ -1,5 +1,17 @@
2016-06-08 Brian Burg <bb...@apple.com>
+ Web Inspector: an error is encountered when trying to delete nodes that have children
+ https://bugs.webkit.org/show_bug.cgi?id=158541
+ <rdar://problem/26705901>
+
+ Reviewed by Timothy Hatcher.
+
+ * UserInterface/Views/DOMTreeElement.js:
+ (WebInspector.DOMTreeElement.prototype.adjustCollapsedRange):
+ Offset into the array variable instead of a variable holding its length.
+
+2016-06-08 Brian Burg <bb...@apple.com>
+
Web Inspector: DOMTreeOutline selection areas should be created and updated lazily
https://bugs.webkit.org/show_bug.cgi?id=158513
<rdar://problem/26689646>
Modified: trunk/Source/WebInspectorUI/UserInterface/Views/DOMTreeElement.js (201842 => 201843)
--- trunk/Source/WebInspectorUI/UserInterface/Views/DOMTreeElement.js 2016-06-08 23:11:23 UTC (rev 201842)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/DOMTreeElement.js 2016-06-08 23:17:11 UTC (rev 201843)
@@ -450,7 +450,7 @@
// In case some nodes from the expanded range were removed, pull some nodes from the collapsed range into the expanded range at the bottom.
for (var i = this.expandedChildCount, limit = Math.min(this.expandedChildrenLimit, totalChildrenCount); i < limit; ++i)
- this.insertChildElement(totalChildrenCount[i], i);
+ this.insertChildElement(visibleChildren[i], i);
var expandedChildCount = this.expandedChildCount;
if (totalChildrenCount > this.expandedChildCount) {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes