[heap] pause observers during mark-compact

[heap] pause observers during mark-compact

Inline allocations performed during mark compact aren't real allocations. They
should not trigger inline-allocation-observer notifications.

R=hpayer@chromium.org, ulan@chromium.org
BUG=

Committed: https://crrev.com/471dd3acdfc72a8cdae92982216d8cb7c36bfde7
Cr-Commit-Position: refs/heads/master@{#32566}

[ofrobots, 2015-11-25 20:13:27]

hpayer@, ulan@ PTAL.

[Hannes Payer (out of office), 2015-11-26 09:55:10]

https://codereview.chromium.org/1465223009/diff/1/src/heap/heap.cc
File src/heap/heap.cc (right):

https://codereview.chromium.org/1465223009/diff/1/src/heap/heap.cc#newcode1388
src/heap/heap.cc:1388: new_space()->PauseInlineAllocationObservers();
A scope would be cleaner for pausing and resuming. WDYT?

[ofrobots, 2015-12-02 06:05:31]

https://codereview.chromium.org/1465223009/diff/1/src/heap/heap.cc
File src/heap/heap.cc (right):

https://codereview.chromium.org/1465223009/diff/1/src/heap/heap.cc#newcode1388
src/heap/heap.cc:1388: new_space()->PauseInlineAllocationObservers();
On 2015/11/26 09:55:10, Hannes Payer wrote:
> A scope would be cleaner for pausing and resuming. WDYT?

Done.

[Hannes Payer (out of office), 2015-12-02 08:09:09]

lgtm

[ofrobots, 2015-12-03 13:59:30]

The CQ bit was checked by ofrobots@google.com

[commit-bot: I haz the power, 2015-12-03 13:59:34]

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

[commit-bot: I haz the power, 2015-12-03 14:20:07]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-03 14:21:02]

Description was changed from

==========
[heap] pause observers during mark-compact

Inline allocations performed during mark compact aren't real allocations. They
should not trigger inline-allocation-observer notifications.

R=hpayer@chromium.org, ulan@chromium.org
BUG=
==========

to

==========
[heap] pause observers during mark-compact

Inline allocations performed during mark compact aren't real allocations. They
should not trigger inline-allocation-observer notifications.

R=hpayer@chromium.org, ulan@chromium.org
BUG=

Committed: https://crrev.com/471dd3acdfc72a8cdae92982216d8cb7c36bfde7
Cr-Commit-Position: refs/heads/master@{#32566}
==========

[commit-bot: I haz the power, 2015-12-03 14:21:03]

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