RESOLVED FIXED 62979
Web Inspector: [Chromium] Improve speed of heap profiles dominators view
https://bugs.webkit.org/show_bug.cgi?id=62979
Summary Web Inspector: [Chromium] Improve speed of heap profiles dominators view
Mikhail Naganov
Reported 2011-06-20 05:19:15 PDT
Currently in snapshot we only have a dominator index, which makes calculation of dominated nodes on the fly expensive. Solved by building a back index.
Attachments
patch (15.95 KB, patch)
2011-06-20 05:22 PDT, Mikhail Naganov
pfeldman: review+
mnaganov: commit-queue-
Mikhail Naganov
Comment 1 2011-06-20 05:22:46 PDT
Mikhail Naganov
Comment 2 2011-06-22 12:48:25 PDT
Manually committed http://trac.webkit.org/changeset/89457 2011-06-20 Mikhail Naganov <mnaganov@chromium.org> Reviewed by Pavel Feldman. Web Inspector: [Chromium] Improve speed of heap profiles dominators view. https://bugs.webkit.org/show_bug.cgi?id=62979 * inspector/front-end/DetailedHeapshotGridNodes.js: (WebInspector.HeapSnapshotDominatorObjectNode.prototype._createProvider): * inspector/front-end/HeapSnapshot.js: (WebInspector.HeapSnapshotArraySlice.prototype.item): (WebInspector.HeapSnapshotArraySlice.prototype.slice): (WebInspector.HeapSnapshot.prototype.dispose): (WebInspector.HeapSnapshot.prototype._dominatedNodesOfNode): (WebInspector.HeapSnapshot.prototype._buildReverseIndex.var): (WebInspector.HeapSnapshot.prototype._buildReverseIndex): (WebInspector.HeapSnapshot.prototype._buildRetainers): (WebInspector.HeapSnapshot.prototype._buildNodeIndex): (WebInspector.HeapSnapshot.prototype._buildDominatedNodes): (WebInspector.HeapSnapshot.prototype._getDominatedIndex): (WebInspector.HeapSnapshot.prototype.createNodesProviderForClass): (WebInspector.HeapSnapshot.prototype.createNodesProviderForDominator): (WebInspector.HeapSnapshotFilteredOrderedIterator): (WebInspector.HeapSnapshotFilteredOrderedIterator.prototype._createIterationOrder): (WebInspector.HeapSnapshotFilteredOrderedIterator.prototype.get length): (WebInspector.HeapSnapshotFilteredOrderedIterator.prototype.serializeNextItems): (WebInspector.HeapSnapshotNodesProvider): * inspector/front-end/HeapSnapshotProxy.js: (WebInspector.HeapSnapshotProxy.prototype.createNodesProviderForDominator):
Note You need to log in before you can comment on or make changes to this bug.