XKB RMLVO splitting.

XKB RMLVO splitting.

Layouts with a variant part (e.g. "us(altgr-intl)") could not be
selected.

BUG=362698
TEST=manual on link_freon

Committed: https://crrev.com/e9a82b8201ec476695935926c4c9dc36adc5a03c
Cr-Commit-Position: refs/heads/master@{#309050}

[kpschoedel, 2014-12-17 21:42:42]

kpschoedel@chromium.org changed reviewers:
+ fengyuan@chromium.org, sadrul@chromium.org, spang@chromium.org

[kpschoedel, 2014-12-17 21:42:42]

kpschoedel@chromium.org changed required reviewers:
+ sadrul@chromium.org

[kpschoedel, 2014-12-17 21:42:43]

fengyuan@, I don't know who defines the supported layout names, but perhaps we
should refactor the layout_name → xkb_rule_names into a separate function with
unit tests for the supported layouts.

[FengYuan, 2014-12-18 05:12:43]

On 2014/12/17 21:42:43, kpschoedel wrote:
> fengyuan@, I don't know who defines the supported layout names, but perhaps we
> should refactor the layout_name → xkb_rule_names into a separate function with
> unit tests for the supported layouts.

The layout names have been existing in manifest for a long time. I can refactor
it and add the unittests for the function.

[FengYuan, 2014-12-18 05:14:11]

lgtm

[alexst (slow to review), 2014-12-18 15:07:30]

alexst@chromium.org changed reviewers:
+ alexst@chromium.org

[alexst (slow to review), 2014-12-18 15:07:31]

lgtm

[spang, 2014-12-18 18:25:29]

The CQ bit was checked by spang@chromium.org

[spang, 2014-12-18 18:25:29]

lgtm

[commit-bot: I haz the power, 2014-12-18 18:26:22]

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

[commit-bot: I haz the power, 2014-12-18 18:26:24]

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

[commit-bot: I haz the power, 2014-12-18 18:26:24]

All required reviewers (with asterisk prefixes) have not yet approved this CL.

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[kpschoedel, 2014-12-18 18:33:17]

kpschoedel@chromium.org changed required reviewers:
+ spang@chromium.org
- sadrul@chromium.org

[kpschoedel, 2014-12-18 18:33:29]

The CQ bit was checked by kpschoedel@chromium.org

[commit-bot: I haz the power, 2014-12-18 18:34:02]

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

[commit-bot: I haz the power, 2014-12-18 19:14:48]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-12-18 19:15:55]

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