dmprof expand command supports to analyze unhooked components

dmprof expand command supports to analyze unhooked components

Currently when analyze the memory usage details of the unhooked components, like unhooked-other, the dmprof expand command does nothing. The purpose of this patch is to enhance 'dmprof expand' command. As unhooked maps have no call stack traces, the result just filters MMAP_LIST items in *.heap files, which belongs to current component specified by the command parameter.

R=kouhei@chromium.org
BUG=416314
Committed: https://crrev.com/0599b1455f1dd12b002dc618df23cada5fec345f
Cr-Commit-Position: refs/heads/master@{#295943}

[kouhei (in TOK), 2014-09-22 07:31:27]

Change looks ok to me. Would you elabolate on description? Thanks.

[wensheng, 2014-09-22 08:28:22]

On 2014/09/22 07:31:27, kouhei wrote:
> Change looks ok to me. Would you elabolate on description? Thanks.

Sorry for no details here.
I have some description on the related issue 416314 and the output format also
can be found on my comment there.

The purpose of this patch is to enhance 'dmprof expand' command.
As unhooked maps have no call stack traces, the result just filters MMAP_LIST
items in *.heap files, which belongs to current component specified by the
command parameter.

I hope the above description can help. Thanks.

[kouhei (in TOK), 2014-09-22 08:29:42]

On 2014/09/22 08:28:22, wensheng wrote:
> On 2014/09/22 07:31:27, kouhei wrote:
> > Change looks ok to me. Would you elabolate on description? Thanks.
> 
> Sorry for no details here.
> I have some description on the related issue 416314 and the output format also
> can be found on my comment there.
> 
> The purpose of this patch is to enhance 'dmprof expand' command.
> As unhooked maps have no call stack traces, the result just filters MMAP_LIST
> items in *.heap files, which belongs to current component specified by the
> command parameter.
> 
> I hope the above description can help. Thanks.

Thanks! Would you add it to the patch description?

[wensheng, 2014-09-22 08:42:24]

On 2014/09/22 08:29:42, kouhei wrote:
> On 2014/09/22 08:28:22, wensheng wrote:
> > On 2014/09/22 07:31:27, kouhei wrote:
> > > Change looks ok to me. Would you elabolate on description? Thanks.
> > 
> > Sorry for no details here.
> > I have some description on the related issue 416314 and the output format
also
> > can be found on my comment there.
> > 
> > The purpose of this patch is to enhance 'dmprof expand' command.
> > As unhooked maps have no call stack traces, the result just filters
MMAP_LIST
> > items in *.heap files, which belongs to current component specified by the
> > command parameter.
> > 
> > I hope the above description can help. Thanks.
> 
> Thanks! Would you add it to the patch description?

Done, please check again.

[kouhei (in TOK), 2014-09-22 09:05:03]

The CQ bit was checked by kouhei@chromium.org

[kouhei (in TOK), 2014-09-22 09:05:03]

lgtm

[commit-bot: I haz the power, 2014-09-22 09:05:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/587193002/1

[commit-bot: I haz the power, 2014-09-22 09:41:42]

Committed patchset #1 (id:1) as 5aeba7e465f9ecc7e3a8429c2f0ba0f8362f9e13

[commit-bot: I haz the power, 2014-09-22 09:42:21]

Patchset 1 (id:??) landed as
https://crrev.com/0599b1455f1dd12b002dc618df23cada5fec345f
Cr-Commit-Position: refs/heads/master@{#295943}