Unify and provide one IsGarbageCollectedType<T> implementation.

Unify and provide one IsGarbageCollectedType<T> implementation.

Phase out the need and use of the older blink::IsGarbageCollected<T>
template, and go with the "marker-based" implementation that WTF
provides. But extended slightly to handle mixins without ambiguity +
it will now insist on T's definition being in scope when used so as to be
able to function reliably.

That latter change requires a few uses of collection types (vectors,
hash maps) to be adjusted so that the full element type of the collection
is in scope when code using the collection is compiled. The reason for
this constraint is that the collection types stringently checks that
Blink GCed objects aren't kept in off-heap collections.

R=
BUG=

Committed: https://crrev.com/1cfb1e02887a0157e3214e3886c39ce0966e1be5
Cr-Commit-Position: refs/heads/master@{#395287}

[sof, 2016-05-22 13:00:01]

Description was changed from

==========
Unify and provide one IsGarbageCollectedType<T> implementation.

Phase out the need and use of the older blink::IsGarbageCollected<T>
template, and go with the "marker-based" implementation that WTF
provides. But extended slightly to handle mixins without ambiguity +
it will now insist on T's definition being in scope when used so as to be
able to function reliably.

That latter change requires a few uses of collection types (vectors,
hash maps) to be adjusted so that full element type of the collection is
in scope when code using the collection is compiled. The reason for
this constraint is that the collection types stringently checks that
Blink GCed objects aren't kept in off-heap collections.

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

to

==========
Unify and provide one IsGarbageCollectedType<T> implementation.

Phase out the need and use of the older blink::IsGarbageCollected<T>
template, and go with the "marker-based" implementation that WTF
provides. But extended slightly to handle mixins without ambiguity +
it will now insist on T's definition being in scope when used so as to be
able to function reliably.

That latter change requires a few uses of collection types (vectors,
hash maps) to be adjusted so that the full element type of the collection
is in scope when code using the collection is compiled. The reason for
this constraint is that the collection types stringently checks that
Blink GCed objects aren't kept in off-heap collections.

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

[sof, 2016-05-22 13:00:27]

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

[sof, 2016-05-22 13:00:28]

please take a look.

[haraken, 2016-05-22 14:35:38]

LGTM

[sof, 2016-05-22 14:37:02]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-05-22 14:37:13]

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

[sof, 2016-05-22 15:52:41]

The CQ bit was unchecked by sigbjornf@opera.com

[sof, 2016-05-22 15:53:12]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-05-22 15:53:18]

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

[commit-bot: I haz the power, 2016-05-22 17:24:55]

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

[commit-bot: I haz the power, 2016-05-22 17:24:56]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sof, 2016-05-22 18:13:27]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-05-22 18:13:31]

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

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

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

[commit-bot: I haz the power, 2016-05-22 20:42:04]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sof, 2016-05-22 20:44:34]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-05-22 20:44:44]

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

[commit-bot: I haz the power, 2016-05-22 23:13:15]

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

[commit-bot: I haz the power, 2016-05-22 23:13:17]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sof, 2016-05-23 04:59:34]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-05-23 04:59:52]

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

[commit-bot: I haz the power, 2016-05-23 06:07:21]

Description was changed from

==========
Unify and provide one IsGarbageCollectedType<T> implementation.

Phase out the need and use of the older blink::IsGarbageCollected<T>
template, and go with the "marker-based" implementation that WTF
provides. But extended slightly to handle mixins without ambiguity +
it will now insist on T's definition being in scope when used so as to be
able to function reliably.

That latter change requires a few uses of collection types (vectors,
hash maps) to be adjusted so that the full element type of the collection
is in scope when code using the collection is compiled. The reason for
this constraint is that the collection types stringently checks that
Blink GCed objects aren't kept in off-heap collections.

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

to

==========
Unify and provide one IsGarbageCollectedType<T> implementation.

Phase out the need and use of the older blink::IsGarbageCollected<T>
template, and go with the "marker-based" implementation that WTF
provides. But extended slightly to handle mixins without ambiguity +
it will now insist on T's definition being in scope when used so as to be
able to function reliably.

That latter change requires a few uses of collection types (vectors,
hash maps) to be adjusted so that the full element type of the collection
is in scope when code using the collection is compiled. The reason for
this constraint is that the collection types stringently checks that
Blink GCed objects aren't kept in off-heap collections.

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

[commit-bot: I haz the power, 2016-05-23 06:07:23]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-23 06:08:21]

Description was changed from

==========
Unify and provide one IsGarbageCollectedType<T> implementation.

Phase out the need and use of the older blink::IsGarbageCollected<T>
template, and go with the "marker-based" implementation that WTF
provides. But extended slightly to handle mixins without ambiguity +
it will now insist on T's definition being in scope when used so as to be
able to function reliably.

That latter change requires a few uses of collection types (vectors,
hash maps) to be adjusted so that the full element type of the collection
is in scope when code using the collection is compiled. The reason for
this constraint is that the collection types stringently checks that
Blink GCed objects aren't kept in off-heap collections.

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

to

==========
Unify and provide one IsGarbageCollectedType<T> implementation.

Phase out the need and use of the older blink::IsGarbageCollected<T>
template, and go with the "marker-based" implementation that WTF
provides. But extended slightly to handle mixins without ambiguity +
it will now insist on T's definition being in scope when used so as to be
able to function reliably.

That latter change requires a few uses of collection types (vectors,
hash maps) to be adjusted so that the full element type of the collection
is in scope when code using the collection is compiled. The reason for
this constraint is that the collection types stringently checks that
Blink GCed objects aren't kept in off-heap collections.

R=
BUG=

Committed: https://crrev.com/1cfb1e02887a0157e3214e3886c39ce0966e1be5
Cr-Commit-Position: refs/heads/master@{#395287}
==========

[commit-bot: I haz the power, 2016-05-23 06:08:22]

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