RESOLVED FIXED 77810
Web Inspector: beautify retaining tree items view 
https://bugs.webkit.org/show_bug.cgi?id=77810
Summary Web Inspector: beautify retaining tree items view
Ilya Tikhonovsky
Reported 2012-02-04 05:41:03 PST
%subj%
Attachments
screenshot (128.22 KB, image/png)
2012-02-04 05:43 PST, Ilya Tikhonovsky 		no flags
Details
Patch (6.12 KB, patch)
2012-02-04 06:35 PST, Ilya Tikhonovsky 		no flags
DetailsFormatted DiffDiff
Patch (4.02 KB, patch)
2012-02-05 00:22 PST, Ilya Tikhonovsky 		yurys: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Ilya Tikhonovsky
Comment 1 2012-02-04 05:43:30 PST
Created attachment 125494 [details] screenshot
Ilya Tikhonovsky
Comment 2 2012-02-04 06:35:31 PST
Created attachment 125496 [details] Patch
Yury Semikhatsky
Comment 3 2012-02-04 23:42:01 PST
Comment on attachment 125496 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=125496&action=review > Source/WebCore/inspector/front-end/DetailedHeapshotGridNodes.js:399 > + " && (" + !this.showRetainingEdges + " || (!edge.isInternal && edge.nodeIndex !== 1))\n" + I don't think we want to omit internal references like 'native' in the retainers path, some of them may be real retainers interesting for the user especially wheh there is no other retainer. I think the condition should be more selective.
Ilya Tikhonovsky
Comment 4 2012-02-05 00:22:22 PST
Created attachment 125519 [details] Patch
Ilya Tikhonovsky
Comment 5 2012-02-05 00:27:25 PST
Committed r106755: <http://trac.webkit.org/changeset/106755>
Note You need to log in before you can comment on or make changes to this bug.