Have ASan ignore ObjectAliveTrait<>::isHeapObjectAlive() applied to mixins

Have ASan ignore ObjectAliveTrait<>::isHeapObjectAlive() applied to mixins

When Heap::willObjectBeLazilySwept<T>() is used on a lazy-sweepable
object where T is a mixin, the mixin's isHeapObjectAlive() virtual will
have to be used to adjust the object pointer to the head of the object.

It is unlikely that a compiler will devirtualize that call hence
mark the isHeapObjectAlive() as ASan ignorable.

R=
BUG=

Committed: https://crrev.com/4d46d2703674fc3377395ca699f6657b0a3192c3
Cr-Commit-Position: refs/heads/master@{#375137}

[sof, 2016-02-11 18:45:22]

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

[sof, 2016-02-11 18:45:22]

please take a look.

[sof, 2016-02-11 19:05:30]

Description was changed from

==========
Have ASan ignore ObjectAliveTrait<>::isHeapObjectAlive() applied to mixins

When Heap::willObjectBeLazilySwept<T>() is used on a lazy-sweepable
object where T is a mixin, the mixin's isHeapObjectAlive() virtual will
have to be used to adjust the object pointer to the head of object.

It is unlikely that a compiler will devirtualize that call hence
mark the isHeapObjectAlive() as ASan ignorable.

R=
BUG=
==========

to

==========
Have ASan ignore ObjectAliveTrait<>::isHeapObjectAlive() applied to mixins

When Heap::willObjectBeLazilySwept<T>() is used on a lazy-sweepable
object where T is a mixin, the mixin's isHeapObjectAlive() virtual will
have to be used to adjust the object pointer to the head of the object.

It is unlikely that a compiler will devirtualize that call hence
mark the isHeapObjectAlive() as ASan ignorable.

R=
BUG=
==========

[haraken, 2016-02-11 23:48:52]

LGTM

[sof, 2016-02-12 06:13:53]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-02-12 06:14:16]

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

[commit-bot: I haz the power, 2016-02-12 06:25:20]

Description was changed from

==========
Have ASan ignore ObjectAliveTrait<>::isHeapObjectAlive() applied to mixins

When Heap::willObjectBeLazilySwept<T>() is used on a lazy-sweepable
object where T is a mixin, the mixin's isHeapObjectAlive() virtual will
have to be used to adjust the object pointer to the head of the object.

It is unlikely that a compiler will devirtualize that call hence
mark the isHeapObjectAlive() as ASan ignorable.

R=
BUG=
==========

to

==========
Have ASan ignore ObjectAliveTrait<>::isHeapObjectAlive() applied to mixins

When Heap::willObjectBeLazilySwept<T>() is used on a lazy-sweepable
object where T is a mixin, the mixin's isHeapObjectAlive() virtual will
have to be used to adjust the object pointer to the head of the object.

It is unlikely that a compiler will devirtualize that call hence
mark the isHeapObjectAlive() as ASan ignorable.

R=
BUG=
==========

[commit-bot: I haz the power, 2016-02-12 06:25:21]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-16 22:42:05]

Description was changed from

==========
Have ASan ignore ObjectAliveTrait<>::isHeapObjectAlive() applied to mixins

When Heap::willObjectBeLazilySwept<T>() is used on a lazy-sweepable
object where T is a mixin, the mixin's isHeapObjectAlive() virtual will
have to be used to adjust the object pointer to the head of the object.

It is unlikely that a compiler will devirtualize that call hence
mark the isHeapObjectAlive() as ASan ignorable.

R=
BUG=
==========

to

==========
Have ASan ignore ObjectAliveTrait<>::isHeapObjectAlive() applied to mixins

When Heap::willObjectBeLazilySwept<T>() is used on a lazy-sweepable
object where T is a mixin, the mixin's isHeapObjectAlive() virtual will
have to be used to adjust the object pointer to the head of the object.

It is unlikely that a compiler will devirtualize that call hence
mark the isHeapObjectAlive() as ASan ignorable.

R=
BUG=

Committed: https://crrev.com/4d46d2703674fc3377395ca699f6657b0a3192c3
Cr-Commit-Position: refs/heads/master@{#375137}
==========

[commit-bot: I haz the power, 2016-02-16 22:42:05]

Patchset 1 (id:??) landed as
https://crrev.com/4d46d2703674fc3377395ca699f6657b0a3192c3
Cr-Commit-Position: refs/heads/master@{#375137}