Do not show multiple instance of disambiguation menu for TextViewWithClickableSpans

Do not show multiple instance of disambiguation menu for TextViewWithClickableSpans

BUG=650852
Committed: https://crrev.com/9491684515e55b01edcb3f254c8836f86b2a6aa5
Cr-Commit-Position: refs/heads/master@{#423724}

[gogerald1, 2016-10-06 17:20:13]

Description was changed from

==========
only show one popmenu

BUG=
==========

to

==========
Do not show multiple instance of popmenu

BUG=650852
==========

[gogerald1, 2016-10-06 17:20:17]

The CQ bit was checked by gogerald@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-06 17:20:35]

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

[gogerald1, 2016-10-06 17:27:17]

Description was changed from

==========
Do not show multiple instance of popmenu

BUG=650852
==========

to

==========
Do not show multiple instance of disambiguation menu

BUG=650852
==========

[gogerald1, 2016-10-06 17:27:25]

Patchset #1 (id:1) has been deleted

[gogerald1, 2016-10-06 17:27:53]

The CQ bit was checked by gogerald@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-06 17:28:19]

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

[gogerald1, 2016-10-06 17:29:52]

Description was changed from

==========
Do not show multiple instance of disambiguation menu

BUG=650852
==========

to

==========
Do not show multiple instance of disambiguation menu for
TextViewWithClickableSpans

BUG=650852
==========

[gogerald1, 2016-10-06 17:30:59]

gogerald@chromium.org changed reviewers:
+ tedchoc@chromium.org

[gogerald1, 2016-10-06 17:31:00]

Hi, PTAL,

[Ted C, 2016-10-06 17:48:13]

lgtm, but what if you wanted 30 dialogs? :-)

[gogerald1, 2016-10-06 18:01:05]

On 2016/10/06 17:48:13, Ted C wrote:
> lgtm, but what if you wanted 30 dialogs? :-)

You mean 30 disambiguation menus? all clickable spans in one
TextViewWithClickableSpans will be displayed in one disambiguation menu, I don't
see why we want multiple identical menus, :-)

[commit-bot: I haz the power, 2016-10-06 18:53:24]

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

[commit-bot: I haz the power, 2016-10-06 18:53:25]

Dry run: This issue passed the CQ dry run.

[gogerald1, 2016-10-06 23:04:19]

The CQ bit was checked by gogerald@chromium.org

[commit-bot: I haz the power, 2016-10-06 23:04:48]

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-10-06 23:17:48]

Description was changed from

==========
Do not show multiple instance of disambiguation menu for
TextViewWithClickableSpans

BUG=650852
==========

to

==========
Do not show multiple instance of disambiguation menu for
TextViewWithClickableSpans

BUG=650852
==========

[commit-bot: I haz the power, 2016-10-06 23:17:50]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-10-06 23:19:56]

Description was changed from

==========
Do not show multiple instance of disambiguation menu for
TextViewWithClickableSpans

BUG=650852
==========

to

==========
Do not show multiple instance of disambiguation menu for
TextViewWithClickableSpans

BUG=650852

Committed: https://crrev.com/9491684515e55b01edcb3f254c8836f86b2a6aa5
Cr-Commit-Position: refs/heads/master@{#423724}
==========

[commit-bot: I haz the power, 2016-10-06 23:19:57]

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