Bug 86102

Summary: Web Inspector: heap snapshot comparison view is broken
Product: WebKit Reporter: Yury Semikhatsky <yurys>
Component: Web Inspector (Deprecated)Assignee: Yury Semikhatsky <yurys>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, webkit.review.bot, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

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.