Reland of [Tracing] Adjust allocation register size for low-end devices

Reland of [Tracing] Adjust allocation register size for low-end devices

Original CL: https://crrev.com/1530583003 by seantopping@chromium.org.

> This patch limits the size of the allocation register and prevents OOM
> errors when running AllocationRegisterTest.OverflowDeathTest on
> low-end devices.
>
> BUG=570242
>
> Committed: https://crrev.com/684e479f9a46580b5d5bdcf58944b832f6317dc1
> Cr-Commit-Position: refs/heads/master@{#365905}

BUG=570242
TBR=primiano@chromium.org,seantopping@chromium.org

Committed: https://crrev.com/895661fb46361b3b272719fd6931ec4c481220be
Cr-Commit-Position: refs/heads/master@{#366081}

[Ruud van Asseldonk, 2015-12-18 10:04:10]

Description was changed from

==========
Reland of [Tracing] Adjust allocation register size for low-end devices

Original CL: https://crrev.com/1530583003 by seantopping@chromium.org.

> This patch limits the size of the allocation register and prevents OOM
> errors when running AllocationRegisterTest.OverflowDeathTest on
> low-end devices.
>
> BUG=570242
>
> Committed: https://crrev.com/684e479f9a46580b5d5bdcf58944b832f6317dc1
> Cr-Commit-Position: refs/heads/master@{#365905}

BUG=570242
==========

to

==========
Reland of [Tracing] Adjust allocation register size for low-end devices

Original CL: https://crrev.com/1530583003 by seantopping@chromium.org.

> This patch limits the size of the allocation register and prevents OOM
> errors when running AllocationRegisterTest.OverflowDeathTest on
> low-end devices.
>
> BUG=570242
>
> Committed: https://crrev.com/684e479f9a46580b5d5bdcf58944b832f6317dc1
> Cr-Commit-Position: refs/heads/master@{#365905}

BUG=570242
TBR=primiano@chromium.org,seantopping@chromium.org
==========

[Ruud van Asseldonk, 2015-12-18 10:36:08]

ruuda@google.com changed reviewers:
+ primiano@chromium.org, seantopping@chromium.org

[Ruud van Asseldonk, 2015-12-18 10:36:09]

https://codereview.chromium.org/1531263004/diff/1/base/trace_event/heap_profi...
File base/trace_event/heap_profiler_allocation_register_unittest.cc (right):

https://codereview.chromium.org/1531263004/diff/1/base/trace_event/heap_profi...
base/trace_event/heap_profiler_allocation_register_unittest.cc:210:
AllocationRegister reg(static_cast<uint32_t>(GetNumCellsPerPage()));
I added this cast to fix the compile on Windows.

[Ruud van Asseldonk, 2015-12-18 10:36:18]

The CQ bit was checked by ruuda@google.com

[commit-bot: I haz the power, 2015-12-18 10:36:37]

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

[commit-bot: I haz the power, 2015-12-18 10:36:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-12-18 10:36:39]

No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Primiano Tucci (use gerrit), 2015-12-18 10:40:10]

LGTM

https://codereview.chromium.org/1531263004/diff/1/base/trace_event/heap_profi...
File base/trace_event/heap_profiler_allocation_register.cc (right):

https://codereview.chromium.org/1531263004/diff/1/base/trace_event/heap_profi...
base/trace_event/heap_profiler_allocation_register.cc:30: 
nit: extra newline?

[Ruud van Asseldonk, 2015-12-18 10:40:33]

The CQ bit was checked by ruuda@google.com

[Ruud van Asseldonk, 2015-12-18 10:40:55]

The CQ bit was unchecked by ruuda@google.com

[Ruud van Asseldonk, 2015-12-18 10:45:17]

The CQ bit was checked by ruuda@google.com

[Ruud van Asseldonk, 2015-12-18 10:45:17]

The patchset sent to the CQ was uploaded after l-g-t-m from
primiano@chromium.org
Link to the patchset: https://codereview.chromium.org/1531263004/#ps20001
(title: "Remove unintended extra newline")

[commit-bot: I haz the power, 2015-12-18 10:45:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1531263004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1531263004/20001

[commit-bot: I haz the power, 2015-12-18 12:03:26]

Description was changed from

==========
Reland of [Tracing] Adjust allocation register size for low-end devices

Original CL: https://crrev.com/1530583003 by seantopping@chromium.org.

> This patch limits the size of the allocation register and prevents OOM
> errors when running AllocationRegisterTest.OverflowDeathTest on
> low-end devices.
>
> BUG=570242
>
> Committed: https://crrev.com/684e479f9a46580b5d5bdcf58944b832f6317dc1
> Cr-Commit-Position: refs/heads/master@{#365905}

BUG=570242
TBR=primiano@chromium.org,seantopping@chromium.org
==========

to

==========
Reland of [Tracing] Adjust allocation register size for low-end devices

Original CL: https://crrev.com/1530583003 by seantopping@chromium.org.

> This patch limits the size of the allocation register and prevents OOM
> errors when running AllocationRegisterTest.OverflowDeathTest on
> low-end devices.
>
> BUG=570242
>
> Committed: https://crrev.com/684e479f9a46580b5d5bdcf58944b832f6317dc1
> Cr-Commit-Position: refs/heads/master@{#365905}

BUG=570242
TBR=primiano@chromium.org,seantopping@chromium.org
==========

[commit-bot: I haz the power, 2015-12-18 12:03:27]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-18 12:04:34]

Description was changed from

==========
Reland of [Tracing] Adjust allocation register size for low-end devices

Original CL: https://crrev.com/1530583003 by seantopping@chromium.org.

> This patch limits the size of the allocation register and prevents OOM
> errors when running AllocationRegisterTest.OverflowDeathTest on
> low-end devices.
>
> BUG=570242
>
> Committed: https://crrev.com/684e479f9a46580b5d5bdcf58944b832f6317dc1
> Cr-Commit-Position: refs/heads/master@{#365905}

BUG=570242
TBR=primiano@chromium.org,seantopping@chromium.org
==========

to

==========
Reland of [Tracing] Adjust allocation register size for low-end devices

Original CL: https://crrev.com/1530583003 by seantopping@chromium.org.

> This patch limits the size of the allocation register and prevents OOM
> errors when running AllocationRegisterTest.OverflowDeathTest on
> low-end devices.
>
> BUG=570242
>
> Committed: https://crrev.com/684e479f9a46580b5d5bdcf58944b832f6317dc1
> Cr-Commit-Position: refs/heads/master@{#365905}

BUG=570242
TBR=primiano@chromium.org,seantopping@chromium.org

Committed: https://crrev.com/895661fb46361b3b272719fd6931ec4c481220be
Cr-Commit-Position: refs/heads/master@{#366081}
==========

[commit-bot: I haz the power, 2015-12-18 12:04:34]

Patchset 2 (id:??) landed as
https://crrev.com/895661fb46361b3b272719fd6931ec4c481220be
Cr-Commit-Position: refs/heads/master@{#366081}