206302 2020-01-15 11:19:04 -0800 Web Inspector: collapsing a virtualized folder in a `WI.TreeOutline` doesn't updated the DOM 2020-01-15 13:42:14 -0800 1 1 1 Unclassified WebKit Web Inspector WebKit Local Build All All RESOLVED FIXED https://webkit.org/ InRadar P2 Normal --- 1 hi hi commit-queue hi inspector-bugzilla-changes timothy webkit-bug-importer oldest_to_newest 1607030 0 hi 2020-01-15 11:19:04 -0800 # STEPS TO REPRODUCE: 1. inspect <https://webkit.org/> 2. set a breakpoint on the first line of `enableScrollableTables` 3. reload the page 4. expand `this` in the Local Variables section of the Scope Chain panel in the details sidebar 5. collapse `this` => there is a bunch of empty space below `this`, when there should be `tables` and the other scope sections NOTE: scrolling fixes the issue 1607033 1 387810 hi 2020-01-15 11:21:44 -0800 Created attachment 387810 Patch 1607137 2 387810 commit-queue 2020-01-15 13:41:55 -0800 Comment on attachment 387810 Patch Clearing flags on attachment: 387810 Committed r254633: <https://trac.webkit.org/changeset/254633> 1607138 3 commit-queue 2020-01-15 13:41:57 -0800 All reviewed patches have been landed. Closing bug. 1607139 4 webkit-bug-importer 2020-01-15 13:42:14 -0800 <rdar://problem/58617549> 387810 2020-01-15 11:21:44 -0800 2020-01-15 13:41:55 -0800 Patch bug-206302-20200115112144.patch text/plain 2747 hi ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cgYi9Tb3VyY2UvV2Vi SW5zcGVjdG9yVUkvQ2hhbmdlTG9nCmluZGV4IDRiZDQ2OWIzMDA0ZDNkNTZiM2VlMzVmZjcyZjNm ZjEwZjBiMWU5N2EuLmNkMDk3ZTg4NDBkZmE2NTAyMjFmNTQwZGE2MTE2Y2NkODMyNzJhNTYgMTAw NjQ0Ci0tLSBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dl Ykluc3BlY3RvclVJL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDIwLTAxLTE1ICBEZXZp biBSb3Vzc28gIDxkcm91c3NvQGFwcGxlLmNvbT4KKworICAgICAgICBXZWIgSW5zcGVjdG9yOiBj b2xsYXBzaW5nIGEgdmlydHVhbGl6ZWQgZm9sZGVyIGluIGEgYFdJLlRyZWVPdXRsaW5lYCBkb2Vz bid0IHVwZGF0ZWQgdGhlIERPTQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93 X2J1Zy5jZ2k/aWQ9MjA2MzAyCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku CisKKyAgICAgICAgKiBVc2VySW50ZXJmYWNlL1ZpZXdzL1RyZWVPdXRsaW5lLmpzOgorICAgICAg ICAoV0kuVHJlZU91dGxpbmUucHJvdG90eXBlLl91cGRhdGVWaXJ0dWFsaXplZEVsZW1lbnRzKToK KyAgICAgICAgV2hlbiBjb2xsYXBzaW5nIGEgY3VycmVudGx5IHZpc2libGUgYFdJLlRyZWVFbGVt ZW50YCwgaXQgd2lsbCBzdGlsbCBiZSBpbiB0aGUgY2FjaGVkIHNldCBvZgorICAgICAgICB2aXNp YmxlIGFuZCBhdHRhY2hlZCBgV0kuVHJlZUVsZW1lbnRgcywgbWVhbmluZyB0aGF0IGBfdXBkYXRl VmlydHVhbGl6ZWRFbGVtZW50c2Agd2lsbCBlYXJseQorICAgICAgICByZXR1cm4gc2luY2UgaXQg dGhpbmtzIHRoYXQgdGhlIHNhbWUgYFdJLlRyZWVFbGVtZW50YCBhcmUgYmVpbmcgc2hvd24uIEFk ZCBhbm90aGVyIGNoZWNrIHRvCisgICAgICAgIGVuc3VyZSB0aGF0IGl0IG9ubHkgdGhpbmtzIHRo YXQgaWYgdGhlIHNhbWUgbnVtYmVyIG9mIGBXSS5UcmVlRWxlbWVudGAgYXJlIHZpc2libGUuCisK IDIwMjAtMDEtMTMgIERldmluIFJvdXNzbyAgPGRyb3Vzc29AYXBwbGUuY29tPgogCiAgICAgICAg IFdlYiBJbnNwZWN0b3I6ICJFbmFibGUgTG9jYWwgT3ZlcnJpZGUiIGFuZCAiRGVsZXRlIExvY2Fs IE92ZXJyaWRlIiBhcmUgZGlzcGxheWVkIHR3aWNlIGluIHRoZSBjb250ZXh0dWFsIG1lbnUKZGlm ZiAtLWdpdCBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9Vc2VySW50ZXJmYWNlL1ZpZXdzL1RyZWVP dXRsaW5lLmpzIGIvU291cmNlL1dlYkluc3BlY3RvclVJL1VzZXJJbnRlcmZhY2UvVmlld3MvVHJl ZU91dGxpbmUuanMKaW5kZXggMDg2Zjc5NTFiMzNmNTUwZDI3N2QzZTg5NTYwYTEwMjc5NTJmZDYx OS4uNWI0MTUwYmE3MGZkYTcwNWIwMWM1YjM1MzMzODQ0MmI1ZGZjMmNmZiAxMDA2NDQKLS0tIGEv U291cmNlL1dlYkluc3BlY3RvclVJL1VzZXJJbnRlcmZhY2UvVmlld3MvVHJlZU91dGxpbmUuanMK KysrIGIvU291cmNlL1dlYkluc3BlY3RvclVJL1VzZXJJbnRlcmZhY2UvVmlld3MvVHJlZU91dGxp bmUuanMKQEAgLTEwNDIsMTEgKzEwNDIsMTQgQEAgV0kuVHJlZU91dGxpbmUgPSBjbGFzcyBUcmVl T3V0bGluZSBleHRlbmRzIFdJLk9iamVjdAogCiAgICAgICAgIC8vIFJlZHJhdyBpZiB3ZSBhcmUg YWJvdXQgdG8gc2Nyb2xsLgogICAgICAgICBpZiAoIXNob3VsZFNjcm9sbCkgewotICAgICAgICAg ICAgLy8gUmVkcmF3IGlmIGFsbCBvZiB0aGUgcHJldmlvdXNseSBjZW50ZXJlZCBgV0kuVHJlZUVs ZW1lbnRgIGFyZSBubyBsb25nZXIgY2VudGVyZWQuCi0gICAgICAgICAgICBpZiAodmlzaWJsZVRy ZWVFbGVtZW50cy5pbnRlcnNlY3RzKHRoaXMuX3ZpcnR1YWxpemVkVmlzaWJsZVRyZWVFbGVtZW50 cykpIHsKLSAgICAgICAgICAgICAgICAvLyBSZWRyYXcgaWYgdGhlcmUgaXMgYSBgV0kuVHJlZUVs ZW1lbnRgIHRoYXQgc2hvdWxkIGJlIHNob3duIHRoYXQgaXNuJ3QgYXR0YWNoZWQuCi0gICAgICAg ICAgICAgICAgaWYgKHZpc2libGVUcmVlRWxlbWVudHMuaXNTdWJzZXRPZih0aGlzLl92aXJ0dWFs aXplZEF0dGFjaGVkVHJlZUVsZW1lbnRzKSkKLSAgICAgICAgICAgICAgICAgICAgcmV0dXJuOwor ICAgICAgICAgICAgLy8gUmVkcmF3IGlmIHRoZXJlIGFyZSBhIGRpZmZlcmVudCBudW1iZXIgb2Yg aXRlbXMgdG8gc2hvdy4KKyAgICAgICAgICAgIGlmICh2aXNpYmxlVHJlZUVsZW1lbnRzLnNpemUg PT09IHRoaXMuX3ZpcnR1YWxpemVkVmlzaWJsZVRyZWVFbGVtZW50cy5zaXplKSB7CisgICAgICAg ICAgICAgICAgLy8gUmVkcmF3IGlmIGFsbCBvZiB0aGUgcHJldmlvdXNseSBjZW50ZXJlZCBgV0ku VHJlZUVsZW1lbnRgIGFyZSBubyBsb25nZXIgY2VudGVyZWQuCisgICAgICAgICAgICAgICAgaWYg KHZpc2libGVUcmVlRWxlbWVudHMuaW50ZXJzZWN0cyh0aGlzLl92aXJ0dWFsaXplZFZpc2libGVU cmVlRWxlbWVudHMpKSB7CisgICAgICAgICAgICAgICAgICAgIC8vIFJlZHJhdyBpZiB0aGVyZSBp cyBhIGBXSS5UcmVlRWxlbWVudGAgdGhhdCBzaG91bGQgYmUgc2hvd24gdGhhdCBpc24ndCBhdHRh Y2hlZC4KKyAgICAgICAgICAgICAgICAgICAgaWYgKHZpc2libGVUcmVlRWxlbWVudHMuaXNTdWJz ZXRPZih0aGlzLl92aXJ0dWFsaXplZEF0dGFjaGVkVHJlZUVsZW1lbnRzKSkKKyAgICAgICAgICAg ICAgICAgICAgICAgIHJldHVybjsKKyAgICAgICAgICAgICAgICB9CiAgICAgICAgICAgICB9CiAg ICAgICAgIH0KIAo=