[SmartText selection] Restore separator between assist and other items

[Smarttext selection] Restore separator between assist and other items

The spec recommends having a separator between an assist item (map,
phone, etc.) and others. The separator is a menu group separator,
we enable it by having a separate menu group for the assist item.

BUG=725693
Review-Url: https://codereview.chromium.org/2903733002
Cr-Commit-Position: refs/heads/master@{#474355}
Committed: https://chromium.googlesource.com/chromium/src/+/78e10e9f196add35245bba8e7b74b1baa7085072

[Tima Vaisburd, 2017-05-23 22:54:40]

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

[Tima Vaisburd, 2017-05-23 22:54:56]

Description was changed from

==========
[Smarttext selection] Restore separator between assist and other items

The spec recommends having a separator between an assist item (map,
phone, etc.) and others. The separator is a menu group separator,
we enable it by having a separate menu group for the assist item.

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

to

==========
[Smarttext selection] Restore separator between assist and other items

The spec recommends having a separator between an assist item (map,
phone, etc.) and others. The separator is a menu group separator,
we enable it by having a separate menu group for the assist item.

BUG=725693
==========

[commit-bot: I haz the power, 2017-05-23 22:55:41]

Dry run: CQ is trying da patch.

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

[Tima Vaisburd, 2017-05-23 22:55:52]

timav@chromium.org changed reviewers:
+ amaralp@chromium.org, boliu@chromium.org

[Tima Vaisburd, 2017-05-23 22:55:53]

PTAL.

[boliu, 2017-05-23 22:57:57]

lgtm

[amaralp, 2017-05-23 23:04:59]

lgtm

[Tima Vaisburd, 2017-05-24 00:21:10]

The CQ bit was unchecked by timav@chromium.org

[Tima Vaisburd, 2017-05-24 00:21:12]

The CQ bit was checked by timav@chromium.org

[Tima Vaisburd, 2017-05-24 00:21:18]

The CQ bit was checked by timav@chromium.org

[commit-bot: I haz the power, 2017-05-24 00:21:25]

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, 2017-05-24 00:32:00]

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

[commit-bot: I haz the power, 2017-05-24 00:32:01]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Tima Vaisburd, 2017-05-24 00:40:12]

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

[Tima Vaisburd, 2017-05-24 00:40:16]

I missed the fact that res/ has separate owners.
Ted, may I ask for an owner review, please?

[Ted C, 2017-05-24 17:47:17]

lgtm

[Tima Vaisburd, 2017-05-24 17:59:42]

The CQ bit was checked by timav@chromium.org

[commit-bot: I haz the power, 2017-05-24 18:00:33]

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, 2017-05-24 18:07:53]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495648782740720, "parent_rev":
"0445935b1f4740e0e283c3fcd5fa7760e9d517b7", "commit_rev":
"78e10e9f196add35245bba8e7b74b1baa7085072"}

[commit-bot: I haz the power, 2017-05-24 18:08:03]

Description was changed from

==========
[Smarttext selection] Restore separator between assist and other items

The spec recommends having a separator between an assist item (map,
phone, etc.) and others. The separator is a menu group separator,
we enable it by having a separate menu group for the assist item.

BUG=725693
==========

to

==========
[Smarttext selection] Restore separator between assist and other items

The spec recommends having a separator between an assist item (map,
phone, etc.) and others. The separator is a menu group separator,
we enable it by having a separate menu group for the assist item.

BUG=725693

Review-Url: https://codereview.chromium.org/2903733002
Cr-Commit-Position: refs/heads/master@{#474355}
Committed:
https://chromium.googlesource.com/chromium/src/+/78e10e9f196add35245bba8e7b74...
==========

[commit-bot: I haz the power, 2017-05-24 18:08:04]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/78e10e9f196add35245bba8e7b74...