Revert of Implement PopupMenu::updateFromElement() for ExternalPopupMenu.

Revert of Implement PopupMenu::updateFromElement() for ExternalPopupMenu. (patchset #2 id:20001 of https://codereview.chromium.org/1605233002/ )

Reason for revert:
caused a regression, crbug.com/579895

Original issue's description:
> Implement PopupMenu::updateFromElement() for ExternalPopupMenu.
>
> It closes a shown popup, and re-show a popup with the latest content.
> This CL affects only OSX and Android.
>
> Implementation:
> A major part of show() was moved to showInternal(), and updateFromElement() calls
> showInternal() later.  showInternal() returns false if it failed to show a popup.
>
> BUG=176241
> TEST=manual; See crbug.com/176241.
>
> Committed: https://crrev.com/f32f9fad19a8366c837e3d10112001c8e7d42606
> Cr-Commit-Position: refs/heads/master@{#370329}

TBR=keishi@chromium.org
NOTRY=true
BUG=176241, 579895
Committed: https://crrev.com/e3f162c974456d915d29c8b98ddda040de1cd12e
Cr-Commit-Position: refs/heads/master@{#370657}

[tkent, 2016-01-21 09:54:21]

Created Revert of Implement PopupMenu::updateFromElement() for
ExternalPopupMenu.

[tkent, 2016-01-21 09:56:03]

Description was changed from

==========
Revert of Implement PopupMenu::updateFromElement() for ExternalPopupMenu.
(patchset #2 id:20001 of https://codereview.chromium.org/1605233002/ )

Reason for revert:
caused a regression, crbug.com/579895

Original issue's description:
> Implement PopupMenu::updateFromElement() for ExternalPopupMenu.
> 
> It closes a shown popup, and re-show a popup with the latest content.
> This CL affects only OSX and Android.
> 
> Implementation:
> A major part of show() was moved to showInternal(), and updateFromElement()
calls
> showInternal() later.  showInternal() returns false if it failed to show a
popup.
> 
> BUG=176241
> TEST=manual; See crbug.com/176241.
> 
> Committed: https://crrev.com/f32f9fad19a8366c837e3d10112001c8e7d42606
> Cr-Commit-Position: refs/heads/master@{#370329}

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

to

==========
Revert of Implement PopupMenu::updateFromElement() for ExternalPopupMenu.
(patchset #2 id:20001 of https://codereview.chromium.org/1605233002/ )

Reason for revert:
caused a regression, crbug.com/579895

Original issue's description:
> Implement PopupMenu::updateFromElement() for ExternalPopupMenu.
> 
> It closes a shown popup, and re-show a popup with the latest content.
> This CL affects only OSX and Android.
> 
> Implementation:
> A major part of show() was moved to showInternal(), and updateFromElement()
calls
> showInternal() later.  showInternal() returns false if it failed to show a
popup.
> 
> BUG=176241
> TEST=manual; See crbug.com/176241.
> 
> Committed: https://crrev.com/f32f9fad19a8366c837e3d10112001c8e7d42606
> Cr-Commit-Position: refs/heads/master@{#370329}

TBR=keishi@chromium.org
NOTRY=true
BUG=176241,579895
==========

[tkent, 2016-01-21 09:56:08]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-01-21 09:56:18]

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

[commit-bot: I haz the power, 2016-01-21 10:01:01]

Committed patchset #1 (id:1)

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

Description was changed from

==========
Revert of Implement PopupMenu::updateFromElement() for ExternalPopupMenu.
(patchset #2 id:20001 of https://codereview.chromium.org/1605233002/ )

Reason for revert:
caused a regression, crbug.com/579895

Original issue's description:
> Implement PopupMenu::updateFromElement() for ExternalPopupMenu.
> 
> It closes a shown popup, and re-show a popup with the latest content.
> This CL affects only OSX and Android.
> 
> Implementation:
> A major part of show() was moved to showInternal(), and updateFromElement()
calls
> showInternal() later.  showInternal() returns false if it failed to show a
popup.
> 
> BUG=176241
> TEST=manual; See crbug.com/176241.
> 
> Committed: https://crrev.com/f32f9fad19a8366c837e3d10112001c8e7d42606
> Cr-Commit-Position: refs/heads/master@{#370329}

TBR=keishi@chromium.org
NOTRY=true
BUG=176241,579895
==========

to

==========
Revert of Implement PopupMenu::updateFromElement() for ExternalPopupMenu.
(patchset #2 id:20001 of https://codereview.chromium.org/1605233002/ )

Reason for revert:
caused a regression, crbug.com/579895

Original issue's description:
> Implement PopupMenu::updateFromElement() for ExternalPopupMenu.
>
> It closes a shown popup, and re-show a popup with the latest content.
> This CL affects only OSX and Android.
>
> Implementation:
> A major part of show() was moved to showInternal(), and updateFromElement()
calls
> showInternal() later.  showInternal() returns false if it failed to show a
popup.
>
> BUG=176241
> TEST=manual; See crbug.com/176241.
>
> Committed: https://crrev.com/f32f9fad19a8366c837e3d10112001c8e7d42606
> Cr-Commit-Position: refs/heads/master@{#370329}

TBR=keishi@chromium.org
NOTRY=true
BUG=176241,579895

Committed: https://crrev.com/e3f162c974456d915d29c8b98ddda040de1cd12e
Cr-Commit-Position: refs/heads/master@{#370657}
==========

[commit-bot: I haz the power, 2016-01-21 10:02:23]

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