[Android] Force showing insertion handle when paste popup activated

[Android] Force showing insertion handle when paste popup activated

An unhandled context menu may result in the showing of the paste popup. However,
this paste popup notification is not well-ordered with respect to other
notifications that determine insertion handle state, namely, position and
region editability updates. Consequently, the paste popup may be shown without
a corresponding insertion handle. Resolve this by forcing an editability and
position update when the paste popup notification is received.

BUG=400304, 400330, 396997
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=287845

[jdduke (slow), 2014-08-05 16:28:41]

aelias@: PTAL.  This is a rather unfortunate workaround.  My plan is to remove
the extra "paste popup" command entirely, instead using the LongPress event +
insertion update to trigger it instead.

[aelias_OOO_until_Jul13, 2014-08-05 20:42:02]

lgtm.  But it would be nice to get to the bottom of the raciness here to
simplify the logic.

[jdduke (slow), 2014-08-05 21:00:40]

On 2014/08/05 20:42:02, aelias wrote:
> lgtm.  But it would be nice to get to the bottom of the raciness here to
> simplify the logic.

Once we enable updates from the compositor, I plan to embed editability into the
(handle metadata) update itself so there's no longer any raciness.

[jdduke (slow), 2014-08-05 21:00:49]

The CQ bit was checked by jdduke@chromium.org

[commit-bot: I haz the power, 2014-08-05 21:03:35]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jdduke@chromium.org/447493002/1

[commit-bot: I haz the power, 2014-08-06 01:23:40]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_chromium_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[jdduke (slow), 2014-08-06 03:29:33]

The CQ bit was unchecked by jdduke@chromium.org

[jdduke (slow), 2014-08-06 15:13:43]

The CQ bit was checked by jdduke@chromium.org

[commit-bot: I haz the power, 2014-08-06 15:14:52]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jdduke@chromium.org/447493002/20001

[commit-bot: I haz the power, 2014-08-06 20:35:08]

Change committed as 287845