Bug 65028 - Web Inspector: [Chromium] Quick clicking on heap objects results in intermixed retaining paths
Summary: Web Inspector: [Chromium] Quick clicking on heap objects results in intermixe...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Mikhail Naganov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-22 07:54 PDT by Mikhail Naganov
Modified: 2011-07-25 05:56 PDT (History)
10 users (show)

See Also:


Attachments
Patch (8.45 KB, patch)
2011-07-22 11:19 PDT, Mikhail Naganov
pfeldman: review+
mnaganov: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mikhail Naganov 2011-07-22 07:54:56 PDT
Take a heap snapshot of a big web app (e.g. gmail). Wait until it is loaded and parsed.
Expand several object types so several object instances are visible at the same time.

Start clicking on different objects to get retaining paths. As a result, retaining paths calculated will be mixed in the paths window (a clear indication is presence of several "Show next 100" buttons).

Clicking on an object should either clean up previous paths and abandon their calculation, or open a new tab for retaining paths to demux calculation results.
Comment 1 Mikhail Naganov 2011-07-22 11:19:20 PDT
Created attachment 101744 [details]
Patch
Comment 2 Mikhail Naganov 2011-07-25 05:56:01 PDT
Manually committed http://trac.webkit.org/changeset/91663


Web Inspector: [Chromium] Fix mixing of retaining path in heap profiler.
https://bugs.webkit.org/show_bug.cgi?id=65028

Reviewed by Pavel Feldman.

* inspector/front-end/DetailedHeapshotView.js:
(WebInspector.HeapSnapshotPathFinderState): Added
(WebInspector.HeapSnapshotRetainingPathsList): Extracted State-specific code