Summary: | Web Inspector: Improve name sorting in HeapSnapshot data grids | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Joseph Pecoraro <joepeck> | ||||||||||
Component: | Web Inspector | Assignee: | Joseph Pecoraro <joepeck> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | bburg, commit-queue, ggaren, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer | ||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||
Version: | WebKit Nightly Build | ||||||||||||
Hardware: | All | ||||||||||||
OS: | All | ||||||||||||
Attachments: |
|
Description
Joseph Pecoraro
2016-11-29 16:00:52 PST
Created attachment 295663 [details]
[PATCH] Proposed Fix
Created attachment 295664 [details]
[IMAGE] Object Graph with improved name sort
Created attachment 295666 [details]
[IMAGE] Instances with improved name sort (object id)
(In reply to comment #4) > Created attachment 295666 [details] > [IMAGE] Instances with improved name sort (object id) Was this screenshot taken before the column rename to "Name"? (In reply to comment #5) > (In reply to comment #4) > > Created attachment 295666 [details] > > [IMAGE] Instances with improved name sort (object id) > > Was this screenshot taken before the column rename to "Name"? Nope, the Instances view keeps the name "Class Name". Only the "Object Graph" one changed to "Name". Do you think it would make sense to change both to be "Name"? (In reply to comment #6) > (In reply to comment #5) > > (In reply to comment #4) > > > Created attachment 295666 [details] > > > [IMAGE] Instances with improved name sort (object id) > > > > Was this screenshot taken before the column rename to "Name"? > > Nope, the Instances view keeps the name "Class Name". Only the "Object > Graph" one changed to "Name". Do you think it would make sense to change > both to be "Name"? I think they should both be "Name", since the column has a mix of class names, property names, etc. Created attachment 295678 [details]
[PATCH] For Landing
Comment on attachment 295678 [details] [PATCH] For Landing Clearing flags on attachment: 295678 Committed r209115: <http://trac.webkit.org/changeset/209115> |