Remove anchor view from container when reset

Remove anchor view from container when reset

Makes sure ScopedAnchorView::Reset() removes the anchor view
from its container view through JNI. This prevents potential
leaks when the destructor fails to do the cleaning job. Not
apparently related to the bug, but it helps avoid any issues
that may happen in the future.

Also deleted a method declaration of |SetWindowAndroid| sneaked
in by mistake.

BUG=635515
Committed: https://crrev.com/0a717f521200898d1fe45c37b351daad1c4a4a32
Cr-Commit-Position: refs/heads/master@{#414154}

[Jinsuk Kim, 2016-08-24 01:25:17]

jinsukkim@chromium.org changed reviewers:
+ dtrainor@chromium.org, sievers@chromium.org

[Jinsuk Kim, 2016-08-24 01:25:18]

Note: The change will have the destructor call delegate_.reset() in addition to
what was being done, which is acceptable.

[David Trainor- moved to gerrit, 2016-08-24 03:18:22]

lgtm

[no sievers, 2016-08-24 18:48:20]

lgtm, thanks!

[no sievers, 2016-08-24 18:48:34]

The CQ bit was checked by sievers@chromium.org

[commit-bot: I haz the power, 2016-08-24 18:49:32]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-24 21:42:26]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-24 21:44:21]

Description was changed from

==========
Remove anchor view from container when reset

Makes sure ScopedAnchorView::Reset() removes the anchor view
from its container view through JNI. This prevents potential
leaks when the destructor fails to do the cleaning job. Not
apparently related to the bug, but it helps avoid any issues
that may happen in the future.

Also deleted a method declaration of |SetWindowAndroid| sneaked
in by mistake.

BUG=635515
==========

to

==========
Remove anchor view from container when reset

Makes sure ScopedAnchorView::Reset() removes the anchor view
from its container view through JNI. This prevents potential
leaks when the destructor fails to do the cleaning job. Not
apparently related to the bug, but it helps avoid any issues
that may happen in the future.

Also deleted a method declaration of |SetWindowAndroid| sneaked
in by mistake.

BUG=635515

Committed: https://crrev.com/0a717f521200898d1fe45c37b351daad1c4a4a32
Cr-Commit-Position: refs/heads/master@{#414154}
==========

[commit-bot: I haz the power, 2016-08-24 21:44:24]

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