RESOLVED FIXED 158584
Web Inspector: heap snapshot comparison view shows nothing if later snapshot is chosen first 
https://bugs.webkit.org/show_bug.cgi?id=158584
Summary Web Inspector: heap snapshot comparison view shows nothing if later snapshot ...
Blaze Burg
Reported 2016-06-09 14:31:45 PDT
Created attachment 280949 [details] Comparing 3 to 5 This is really confusing and looks like a bug. Maybe it should always sort the comparison to put earlier first. I can't think of a use case for actually showing the diff in reverse.
Attachments
Comparing 3 to 5 (113.60 KB, image/png)
2016-06-09 14:31 PDT, Blaze Burg 		no flags
Details
Comparing 5 to 3 (83.45 KB, image/png)
2016-06-09 14:32 PDT, Blaze Burg 		no flags
Details
[Patch] Proposed Fix (2.78 KB, patch)
2016-06-10 14:33 PDT, Matt Baker 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Blaze Burg
Comment 1 2016-06-09 14:32:04 PDT
Created attachment 280950 [details] Comparing 5 to 3
Radar WebKit Bug Importer
Comment 2 2016-06-09 14:32:40 PDT
<rdar://problem/26727849>
Radar WebKit Bug Importer
Comment 3 2016-06-09 14:37:07 PDT
<rdar://problem/26727887>
Matt Baker
Comment 4 2016-06-10 14:33:00 PDT
Created attachment 281047 [details] [Patch] Proposed Fix
WebKit Commit Bot
Comment 5 2016-06-10 16:04:21 PDT
Comment on attachment 281047 [details] [Patch] Proposed Fix Clearing flags on attachment: 281047 Committed r201949: <http://trac.webkit.org/changeset/201949>
WebKit Commit Bot
Comment 6 2016-06-10 16:04:25 PDT
All reviewed patches have been landed. Closing bug.
Joseph Pecoraro
Comment 7 2016-06-10 17:26:29 PDT
Comment on attachment 281047 [details] [Patch] Proposed Fix Nice fix. This is exactly what I would have done.
Blaze Burg
Comment 8 2016-06-10 19:10:04 PDT
(In reply to comment #7) > Comment on attachment 281047 [details] > [Patch] Proposed Fix > > Nice fix. This is exactly what I would have done. Wohoo! Thanks.
Note You need to log in before you can comment on or make changes to this bug.