Bug 86102 - Web Inspector: heap snapshot comparison view is broken
Summary: Web Inspector: heap snapshot comparison view is broken
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: Yury Semikhatsky
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-10 08:29 PDT by Yury Semikhatsky
Modified: 2012-05-10 13:23 PDT (History)
11 users (show)

See Also:

Attachments
Patch (1.60 KB, patch)
2012-05-10 09:33 PDT, Yury Semikhatsky 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Yury Semikhatsky 2012-05-10 08:29:11 PDT 
1. Take 1 heap snapshot, wait for the summary view to open.
2. Take another heap snapshot, wait for the summary view to open.
3. Select "Comparison" view.

Result:
Nothing happens.

Note that after frontend reopening if you open the second snapshot and select "Comparison" view the diff will show up.
Comment 1 Yury Semikhatsky 2012-05-10 08:47:14 PDT 
This is a regression after http://trac.webkit.org/changeset/116218 (early return from WebInspector.HeapProfileHeader.prototype.load)
Comment 2 Yury Semikhatsky 2012-05-10 09:33:01 PDT 
Created attachment 141184 [details]
Patch
Comment 3 WebKit Review Bot 2012-05-10 13:23:41 PDT 
Comment on attachment 141184 [details]
Patch

Clearing flags on attachment: 141184

Committed r116681: <http://trac.webkit.org/changeset/116681>
Comment 4 WebKit Review Bot 2012-05-10 13:23:46 PDT 
All reviewed patches have been landed.  Closing bug.