[heap] make inline-allocation-observers precise

[heap] make inline-allocation-observers precise

Now that we no longer require AllocationInfo::limit to be aligned [1], we can do
more accurate inline-allocation-observation. This lets us get notified when the
next allocation that crosses the step-size boundary is allocated.

Fixed the test-cases. They make significantly more sense now given the step
sizes and the number of times we get notifications. For example, with a step
size of 512, an allocation of 16kb results in 32 notifications instead of 30
now.

[1] https://codereview.chromium.org/1444883003

R=hpayer@chromium.org
BUG=

Committed: https://crrev.com/0514fa204a39682fbc4ec7de19ac03aac08ee462
Cr-Commit-Position: refs/heads/master@{#32091}

[Hannes Payer (out of office), 2015-11-17 14:41:31]

lgtm, just comment nits

https://codereview.chromium.org/1448913003/diff/20001/src/heap/spaces.h
File src/heap/spaces.h (right):

https://codereview.chromium.org/1448913003/diff/20001/src/heap/spaces.h#newco...
src/heap/spaces.h:2525: // least step_size byte have been allocated.
byte => bytes

https://codereview.chromium.org/1448913003/diff/20001/src/heap/spaces.h#newco...
src/heap/spaces.h:2531: // Step method is only called once at least than
step_size bytes have been
This sentence seems to be broken. Do we still need it?

[ofrobots, 2015-11-17 19:47:22]

The CQ bit was checked by ofrobots@google.com

[ofrobots, 2015-11-17 19:47:22]

The patchset sent to the CQ was uploaded after l-g-t-m from hpayer@chromium.org
Link to the patchset: https://codereview.chromium.org/1448913003/#ps40001
(title: "fix comment nits")

[commit-bot: I haz the power, 2015-11-17 19:47:29]

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

[commit-bot: I haz the power, 2015-11-17 20:21:57]

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

[commit-bot: I haz the power, 2015-11-17 20:21:57]

Try jobs failed on following builders:
  v8_linux_dbg on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg/builds/10858)

[ofrobots, 2015-11-17 21:20:12]

On 2015/11/17 20:21:57, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   v8_linux_dbg on tryserver.v8 (JOB_FAILED,
> http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg/builds/10858)

Rebased.

[ofrobots, 2015-11-17 21:20:17]

The CQ bit was checked by ofrobots@google.com

[ofrobots, 2015-11-17 21:20:18]

The patchset sent to the CQ was uploaded after l-g-t-m from hpayer@chromium.org
Link to the patchset: https://codereview.chromium.org/1448913003/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2015-11-17 21:20:21]

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

[commit-bot: I haz the power, 2015-11-17 21:51:36]

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

[commit-bot: I haz the power, 2015-11-17 21:51:37]

Try jobs failed on following builders:
  v8_linux_dbg on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg/builds/10864)

[ofrobots, 2015-11-17 22:02:01]

On 2015/11/17 21:51:37, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   v8_linux_dbg on tryserver.v8 (JOB_FAILED,
> http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg/builds/10864)

hpayer@: I needed to remove an additional instance of the 'limit should be
assigned assertion'. I think this requires another look at spaces.h before I
merge. Thanks.

[Hannes Payer (out of office), 2015-11-18 08:58:02]

lgtm

[ofrobots, 2015-11-18 15:27:08]

The CQ bit was checked by ofrobots@google.com

[commit-bot: I haz the power, 2015-11-18 15:27:18]

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

[commit-bot: I haz the power, 2015-11-18 15:54:28]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-11-18 15:55:12]

Patchset 5 (id:??) landed as
https://crrev.com/0514fa204a39682fbc4ec7de19ac03aac08ee462
Cr-Commit-Position: refs/heads/master@{#32091}