Bug 3613 - heapshot tool UI is pretty bad..
Summary: heapshot tool UI is pretty bad..
Status: RESOLVED NOT_ON_ROADMAP
Alias: None
Product: iOS
Classification: Xamarin
Component: Tools ()
Version: 5.0
Hardware: PC Mac OS
: --- normal
Target Milestone: Untriaged
Assignee: Bugzilla
URL:
Depends on: 2553
Blocks:
  Show dependency tree
 
Reported: 2012-02-23 12:05 UTC by Chris Toshok
Modified: 2015-03-31 13:55 UTC (History)
2 users (show)

Tags:
Is this bug a regression?: ---
Last known good build:

Notice (2018-05-24): bugzilla.xamarin.com is now in read-only mode.

Please join us on Visual Studio Developer Community and in the Xamarin and Mono organizations on GitHub to continue tracking issues. Bugzilla will remain available for reference in read-only mode. We will continue to work on open Bugzilla bugs, copy them to the new locations as needed for follow-up, and add the new items under Related Links.

Our sincere thanks to everyone who has contributed on this bug tracker over the years. Thanks also for your understanding as we make these adjustments and improvements for the future.


Please create a new report on Developer Community or GitHub with your current version information, steps to reproduce, and relevant error messages or log files if you are hitting an issue that looks similar to this resolved bug and you do not yet see a matching new report.

Related Links:
Status:
RESOLVED NOT_ON_ROADMAP

Description Chris Toshok 2012-02-23 12:05:04 UTC
I know this is the same UI that existed before, but it's always been bad.

here's a dump from IRC, since I'm pretty sure I was talking to the void.. :)


[08:26:59] <toshok> does generating a heapshot run the collector at all?

[08:34:16] <toshok> any reason why the profiler isn't under the Run menu?
[08:34:23] <toshok> strange to see it under Project
[08:35:22] <toshok> and all sorts of pretty terrible UI going on in the heapshot window
[08:35:33] <toshok> 1) not resizable unless you're showing the gtktable view
[08:36:04] <toshok> 2) why do i have to select the heapshot and then switch to the summary tab to get info about the heapshot?  that should be in columns on the LHS
[08:36:18] <toshok> 3) by default all the columns except type aren't visible at all
[08:36:24] <toshok> 4) soooooo many <Unknown>s
[08:38:16] <toshok> 5) pretty often the window gets confused when i click/drag in the title bar and starts resizing the window instead of moving it.  well, actually the window moves and then snaps back to the old location and resizes some

[08:40:27] <toshok> and finally: 6) the columns are way way way unhelpful.  Root Mem?  Root Refs?  nobody cares about that.  the UI reminds me of gtop.  just because you have the info doesn't mean it should be displayed :)

[08:41:21] <toshok> oh wait, i have a 7) why is there a tooltip on the type column?  it's already showing exactly the same info that the tooltip provides :)

[08:54:40] <toshok> oh oh, one more thing:  8) there should be a "Hide/Show framework classes" toggle
[08:54:58] <toshok> because I don't really care about this UTF8Decoder that is alive
[08:55:28] <toshok> or the 4 System.Text.RegularExpressions.MRUList.Node :)
Comment 1 Rolf Bjarne Kvinge [MSFT] 2012-02-23 16:26:03 UTC
4) is bug #2553
Comment 2 Rolf Bjarne Kvinge [MSFT] 2012-02-23 16:27:43 UTC
Taking a heapshot does cause a collection (it's actually how the heapshot is triggered in the first place).
Comment 3 Rolf Bjarne Kvinge [MSFT] 2012-02-23 16:30:18 UTC
And the only reason the menu entry is under Project instead of Run is because it was easier to do it that way (and nobody voiced a strong opinion either way).
Comment 4 Rolf Bjarne Kvinge [MSFT] 2015-03-31 13:55:52 UTC
HeapShot will be replaced with a better profiler (with a much better UI).