Summary: | Web Inspector: heap snapshot: implement Distance column in Object's retaining tree. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ilya Tikhonovsky <loislo> | ||||||
Component: | Web Inspector (Deprecated) | Assignee: | Ilya Tikhonovsky <loislo> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 78411 | ||||||||
Attachments: |
|
Description
Ilya Tikhonovsky
2012-02-08 06:41:08 PST
Created attachment 126076 [details]
Patch
I'm not sure that we need to see this column in the UI. It doesn't provide much additional information about the retainers. Comment on attachment 126076 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=126076&action=review > Source/WebCore/inspector/front-end/DetailedHeapshotGridNodes.js:436 > + distanceToWindow: ["distanceToWindow", sortAscending, "_name", true] Please make sure sorting by the distance column works if there are expanded entries. r- for this. Created attachment 126087 [details]
Patch
Comment on attachment 126087 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=126087&action=review > Source/WebCore/inspector/front-end/HeapSnapshot.js:946 > + this._distancesToWindow = new Array(this.nodeCount); Use {} instead of array? > Source/WebCore/inspector/front-end/HeapSnapshot.js:969 > + var node = list[index++]; // shift generates too much garbage. Why not just use shift here? Comment on attachment 126087 [details] Patch Clearing flags on attachment: 126087 Committed r107106: <http://trac.webkit.org/changeset/107106> All reviewed patches have been landed. Closing bug. Comment on attachment 126087 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=126087&action=review > Source/WebCore/inspector/front-end/HeapSnapshot.js:978 > + if (typeof this._distancesToWindow[childNode.nodeIndex] !== "undefined") Drive-by: "if (childNode.nodeIndex in this._distancesToWindow)" is shorter |