Revert of Use PhantomReference for CleanupReference

Revert of Use PhantomReference for CleanupReference (patchset #1 id:1 of https://codereview.chromium.org/1802233002/ )

Reason for revert:
Found out that PhantomReference can survive (ie not get enqueued) a GC pass. Tested on Android M.

This would cause unacceptable memory regressions for apps that relies on gc to clean up webview.

Original issue's description:
> android: Use PhantomReference for CleanupReference
>
> PhantomReference is less of a burden on dalvik/art gc systems.
>
> Committed: https://crrev.com/9b1b3599500eca2b8d664714edf6b6771a789123
> Cr-Commit-Position: refs/heads/master@{#381173}

TBR=dtrainor@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.

Committed: https://crrev.com/43f4806331fd76d86e68eea634f2d5ae8cd39cc7
Cr-Commit-Position: refs/heads/master@{#381817}

[boliu, 2016-03-17 19:15:03]

Created Revert of Use PhantomReference for CleanupReference

[commit-bot: I haz the power, 2016-03-17 19:15:20]

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

[commit-bot: I haz the power, 2016-03-17 19:31:55]

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

[commit-bot: I haz the power, 2016-03-17 20:02:01]

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

[commit-bot: I haz the power, 2016-03-17 20:31:44]

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

[commit-bot: I haz the power, 2016-03-17 21:02:01]

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

[commit-bot: I haz the power, 2016-03-17 21:31:54]

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

[David Trainor- moved to gerrit, 2016-03-17 21:56:11]

lgtm

[commit-bot: I haz the power, 2016-03-17 22:02:24]

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

[commit-bot: I haz the power, 2016-03-17 22:32:08]

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

[commit-bot: I haz the power, 2016-03-17 22:43:28]

Description was changed from

==========
Revert of Use PhantomReference for CleanupReference (patchset #1 id:1 of
https://codereview.chromium.org/1802233002/ )

Reason for revert:
Found out that PhantomReference can survive (ie not get enqueued) a GC pass.
Tested on Android M.

This would cause unacceptable memory regressions for apps that relies on gc to
clean up webview.

Original issue's description:
> android: Use PhantomReference for CleanupReference
> 
> PhantomReference is less of a burden on dalvik/art gc systems.
> 
> Committed: https://crrev.com/9b1b3599500eca2b8d664714edf6b6771a789123
> Cr-Commit-Position: refs/heads/master@{#381173}

TBR=dtrainor@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

to

==========
Revert of Use PhantomReference for CleanupReference (patchset #1 id:1 of
https://codereview.chromium.org/1802233002/ )

Reason for revert:
Found out that PhantomReference can survive (ie not get enqueued) a GC pass.
Tested on Android M.

This would cause unacceptable memory regressions for apps that relies on gc to
clean up webview.

Original issue's description:
> android: Use PhantomReference for CleanupReference
> 
> PhantomReference is less of a burden on dalvik/art gc systems.
> 
> Committed: https://crrev.com/9b1b3599500eca2b8d664714edf6b6771a789123
> Cr-Commit-Position: refs/heads/master@{#381173}

TBR=dtrainor@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

[commit-bot: I haz the power, 2016-03-17 22:43:29]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-17 22:45:55]

Description was changed from

==========
Revert of Use PhantomReference for CleanupReference (patchset #1 id:1 of
https://codereview.chromium.org/1802233002/ )

Reason for revert:
Found out that PhantomReference can survive (ie not get enqueued) a GC pass.
Tested on Android M.

This would cause unacceptable memory regressions for apps that relies on gc to
clean up webview.

Original issue's description:
> android: Use PhantomReference for CleanupReference
> 
> PhantomReference is less of a burden on dalvik/art gc systems.
> 
> Committed: https://crrev.com/9b1b3599500eca2b8d664714edf6b6771a789123
> Cr-Commit-Position: refs/heads/master@{#381173}

TBR=dtrainor@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

to

==========
Revert of Use PhantomReference for CleanupReference (patchset #1 id:1 of
https://codereview.chromium.org/1802233002/ )

Reason for revert:
Found out that PhantomReference can survive (ie not get enqueued) a GC pass.
Tested on Android M.

This would cause unacceptable memory regressions for apps that relies on gc to
clean up webview.

Original issue's description:
> android: Use PhantomReference for CleanupReference
>
> PhantomReference is less of a burden on dalvik/art gc systems.
>
> Committed: https://crrev.com/9b1b3599500eca2b8d664714edf6b6771a789123
> Cr-Commit-Position: refs/heads/master@{#381173}

TBR=dtrainor@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.

Committed: https://crrev.com/43f4806331fd76d86e68eea634f2d5ae8cd39cc7
Cr-Commit-Position: refs/heads/master@{#381817}
==========

[commit-bot: I haz the power, 2016-03-17 22:45:55]

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