Make some foo.h headers usable without foo-inl.h header.

Make some foo.h headers usable without foo-inl.h header.

This CL is a pure refactoring that makes an empty compilation unit
including just "foo.h" but not "foo-inl.h" compile without warnings or
errors. This is needed to further reduce the header dependency tangle.

This realizes above state for "spaces.h" and "mark-compact.h".

R=hpayer@chromium.org

Committed: https://crrev.com/16f96fde6b935c5f9a9d031f5df9809a814242ce
Cr-Commit-Position: refs/heads/master@{#30171}

[Michael Starzinger, 2015-08-13 16:32:15]

mstarzinger@chromium.org changed reviewers:
+ mlippautz@chromium.org

[Michael Starzinger, 2015-08-13 16:32:15]

mstarzinger@chromium.org changed required reviewers:
+ hpayer@chromium.org

[Michael Starzinger, 2015-08-13 16:32:16]

Hannes: PTAL.
Mike: FYI.

[Hannes Payer (out of office), 2015-08-14 05:05:48]

https://codereview.chromium.org/1288413002/diff/1/src/heap/mark-compact.h
File src/heap/mark-compact.h (right):

https://codereview.chromium.org/1288413002/diff/1/src/heap/mark-compact.h#new...
src/heap/mark-compact.h:208: void DecrementLiveBytes(HeapObject* object);
This method does not really belong to MarkingDeque. We should probably have a
wrapper in mark-compact for PushBlack and UnshiftBlack that takes care of the
actual pushing and the correct accounting of live bytes. That would
significantly reduce the call sites of MemoryChunk::IncrementLiveBytesFromGC.

[Michael Starzinger, 2015-08-14 07:56:23]

https://codereview.chromium.org/1288413002/diff/1/src/heap/mark-compact.h
File src/heap/mark-compact.h (right):

https://codereview.chromium.org/1288413002/diff/1/src/heap/mark-compact.h#new...
src/heap/mark-compact.h:208: void DecrementLiveBytes(HeapObject* object);
On 2015/08/14 05:05:48, Hannes Payer wrote:
> This method does not really belong to MarkingDeque. We should probably have a
> wrapper in mark-compact for PushBlack and UnshiftBlack that takes care of the
> actual pushing and the correct accounting of live bytes. That would
> significantly reduce the call sites of MemoryChunk::IncrementLiveBytesFromGC.

Acknowledged. Yes, you are right. As per offline discussion, I have taken this
part out of the CL and will do it in a separate CL.

[Hannes Payer (out of office), 2015-08-14 08:26:29]

lgtm

[Michael Starzinger, 2015-08-14 08:46:34]

The CQ bit was checked by mstarzinger@chromium.org

[commit-bot: I haz the power, 2015-08-14 08:46:45]

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

[commit-bot: I haz the power, 2015-08-14 08:48:16]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-08-14 08:48:26]

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