Oilpan: expose near-finalized predicate as Heap::willObjectBeLazilySwept()

Oilpan: expose near-finalized predicate as Heap::willObjectBeLazilySwept()

Provide an inverted, and more focused version of the previous
isFinalizedObjectAlive() predicate. willObjectBeLazilySwept() returns true
iff an object reference has been slated for finalization by sweeping,
but hasn't been yet.

R=haraken
BUG=448392
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=188804

[sof, 2015-01-22 09:19:57]

sigbjornf@opera.com changed reviewers:
+ oilpan-reviews@chromium.org

[sof, 2015-01-22 09:19:58]

Please take a look.

[haraken, 2015-01-22 09:29:29]

haraken@chromium.org changed reviewers:
+ haraken@chromium.org

[haraken, 2015-01-22 09:29:30]

LGTM

https://codereview.chromium.org/858363005/diff/1/Source/platform/heap/Heap.h
File Source/platform/heap/Heap.h (right):

https://codereview.chromium.org/858363005/diff/1/Source/platform/heap/Heap.h#...
Source/platform/heap/Heap.h:861: // to be in; willObjectBeLazilySwept() has
undefined behavior if passed

Can we assert this by checking header->checkHeader()?

[sof, 2015-01-22 09:56:23]

https://codereview.chromium.org/858363005/diff/1/Source/platform/heap/Heap.h
File Source/platform/heap/Heap.h (right):

https://codereview.chromium.org/858363005/diff/1/Source/platform/heap/Heap.h#...
Source/platform/heap/Heap.h:861: // to be in; willObjectBeLazilySwept() has
undefined behavior if passed
On 2015/01/22 09:29:29, haraken wrote:
> 
> Can we assert this by checking header->checkHeader()?

That would be helpful, added.

[sof, 2015-01-22 10:45:09]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2015-01-22 10:45:23]

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

[commit-bot: I haz the power, 2015-01-22 11:41:25]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=188804

[sof, 2015-01-22 13:21:25]

On 2015/01/22 09:56:23, sof wrote:
> https://codereview.chromium.org/858363005/diff/1/Source/platform/heap/Heap.h
> File Source/platform/heap/Heap.h (right):
> 
>
https://codereview.chromium.org/858363005/diff/1/Source/platform/heap/Heap.h#...
> Source/platform/heap/Heap.h:861: // to be in; willObjectBeLazilySwept() has
> undefined behavior if passed
> On 2015/01/22 09:29:29, haraken wrote:
> > 
> > Can we assert this by checking header->checkHeader()?
> 
> That would be helpful, added.

That won't work out readily for mixins. As isMarked() already checks for this, I
will remove.