[Android] Move weak ref for PopupWindowTouchHandleDrawable to native

[Android] Move weak ref for PopupWindowTouchHandleDrawable to native

The current weak reference wiring for PopupWindowTouchHandleDrawable is awkward
and fragile. Move the weak ref accounting to its native object counterpart,
using the JNI boundary as a consistent reference lifetime checkpoint.

BUG=528365
Committed: https://crrev.com/7a60160df059c5031f1d6d040b2779061dc20d9c
Cr-Commit-Position: refs/heads/master@{#348809}

[jdduke (slow), 2015-09-14 20:42:15]

jdduke@chromium.org changed reviewers:
+ aurimas@chromium.org

[jdduke (slow), 2015-09-14 20:42:16]

OK, here's patch #2 of the cleanup (one more coming... let me know if you want
me to find a different reviewer =/).

[aurimas (slooooooooow), 2015-09-14 23:20:53]

LGTM, this makes the life time of the Java object much easier to understand.

[jdduke (slow), 2015-09-14 23:46:12]

The CQ bit was checked by jdduke@chromium.org

[commit-bot: I haz the power, 2015-09-14 23:46:47]

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

[commit-bot: I haz the power, 2015-09-14 23:49:04]

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

[commit-bot: I haz the power, 2015-09-14 23:49:04]

Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[jdduke (slow), 2015-09-15 00:13:18]

The CQ bit was checked by jdduke@chromium.org

[jdduke (slow), 2015-09-15 00:13:18]

The patchset sent to the CQ was uploaded after l-g-t-m from aurimas@chromium.org
Link to the patchset: https://codereview.chromium.org/1341013002/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2015-09-15 00:14:18]

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

[commit-bot: I haz the power, 2015-09-15 01:32:49]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-09-15 01:33:33]

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

[commit-bot: I haz the power, 2015-09-23 12:41:05]

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