PartitionDumpStats(): reduce stack consumption for 'light' reporting.

PartitionDumpStats(): reduce stack consumption for 'light' reporting.

Following r456291's lead, avoid putting a large array on the stack for
light stats reporting in PartitionDumpStats(), where it isn't made use
of. This avoids running into stack pressure when sampling memory use
in production builds (on Windows, in particular.)

R=haraken
BUG=704894
Review-Url: https://codereview.chromium.org/2771033006
Cr-Commit-Position: refs/heads/master@{#459498}
Committed: https://chromium.googlesource.com/chromium/src/+/06fe872f1501d6ea1f79ce592555da1d766c0a7a

[sof, 2017-03-24 14:48:45]

The CQ bit was checked by sigbjornf@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-24 14:49:29]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sof, 2017-03-24 14:59:34]

sigbjornf@opera.com changed reviewers:
+ haraken@chromium.org, palmer@chromium.org

[sof, 2017-03-24 14:59:35]

please take a look.

https://codereview.chromium.org/2741853007 applied to PartitionDumpStats() (at
least that's the intent.)

Will undo the WebKit/Source/ change in ps#2 once the trybots have run
webkit_tests.

[haraken, 2017-03-24 15:15:39]

LGTM

[sof, 2017-03-24 16:26:14]

Description was changed from

==========
PartitionDumpStats(): reduce stack consumption for 'light' reporting.

Following r456291's lead, avoid putting a large array on the stack for
light stats reporting in PartitionDumpStats(), where it isn't made use
of. This avoids running into stack pressure when sampling memory use
in production builds (on Windows, in particular.)

R=
BUG=704894
==========

to

==========
PartitionDumpStats(): reduce stack consumption for 'light' reporting.

Following r456291's lead, avoid putting a large array on the stack for
light stats reporting in PartitionDumpStats(), where it isn't made use
of. This avoids running into stack pressure when sampling memory use
in production builds (on Windows, in particular.)

R=haraken
BUG=704894
==========

[sof, 2017-03-24 16:26:31]

The CQ bit was checked by sigbjornf@opera.com

[sof, 2017-03-24 16:26:31]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2771033006/#ps40001
(title: "undo ps#2")

[commit-bot: I haz the power, 2017-03-24 16:27:08]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-24 18:54:02]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1490372791184440,
"parent_rev": "2736655f73f048c36e9deeda5477ed812dbcac1e", "commit_rev":
"06fe872f1501d6ea1f79ce592555da1d766c0a7a"}

[commit-bot: I haz the power, 2017-03-24 18:55:08]

Description was changed from

==========
PartitionDumpStats(): reduce stack consumption for 'light' reporting.

Following r456291's lead, avoid putting a large array on the stack for
light stats reporting in PartitionDumpStats(), where it isn't made use
of. This avoids running into stack pressure when sampling memory use
in production builds (on Windows, in particular.)

R=haraken
BUG=704894
==========

to

==========
PartitionDumpStats(): reduce stack consumption for 'light' reporting.

Following r456291's lead, avoid putting a large array on the stack for
light stats reporting in PartitionDumpStats(), where it isn't made use
of. This avoids running into stack pressure when sampling memory use
in production builds (on Windows, in particular.)

R=haraken
BUG=704894

Review-Url: https://codereview.chromium.org/2771033006
Cr-Commit-Position: refs/heads/master@{#459498}
Committed:
https://chromium.googlesource.com/chromium/src/+/06fe872f1501d6ea1f79ce592555...
==========

[commit-bot: I haz the power, 2017-03-24 18:55:10]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/06fe872f1501d6ea1f79ce592555...