chromeos: Remove the last IME methods from ash::SystemTrayDelegate

chromeos: Remove the last IME methods from ash::SystemTrayDelegate

For mustash the code in ash will run in its own process. We are eliminating
ash to chrome delegate methods.

Move the remaining IME methods out of SystemTrayDelegateChromeOS into
utility functions in ash::ime_util. Those functions can call into
//ui/base/ime/chromeos directly.

BUG=724305
TEST=ash_unittests, manually test IME shelf menu and system tray menu

[James Cook, 2017-06-01 16:35:38]

Description was changed from

==========
WIP: chromeos: Remove the last IME methods from ash::SystemTrayDelegate

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

to

==========
chromeos: Remove the last IME methods from ash::SystemTrayDelegate

For mustash the code in ash will run in its own process. We are eliminating
ash to chrome delegate methods.

Move the remaining IME methods out of SystemTrayDelegateChromeOS into
utility functions in ash::ime_util. Those functions can call into
//ui/base/ime/chromeos directly.

BUG=724305
TEST=ash_unittests, manually test IME shelf menu and system tray menu
==========

[James Cook, 2017-06-01 16:40:22]

Description was changed from

==========
chromeos: Remove the last IME methods from ash::SystemTrayDelegate

For mustash the code in ash will run in its own process. We are eliminating
ash to chrome delegate methods.

Move the remaining IME methods out of SystemTrayDelegateChromeOS into
utility functions in ash::ime_util. Those functions can call into
//ui/base/ime/chromeos directly.

BUG=724305
TEST=ash_unittests, manually test IME shelf menu and system tray menu
==========

to

==========
chromeos: Remove the last IME methods from ash::SystemTrayDelegate

For mustash the code in ash will run in its own process. We are eliminating
ash to chrome delegate methods.

Move the remaining IME methods out of SystemTrayDelegateChromeOS into
utility functions in ash::ime_util. Those functions can call into
//ui/base/ime/chromeos directly.

BUG=724305
TEST=ash_unittests, manually test IME shelf menu and system tray menu
==========

[James Cook, 2017-06-01 16:42:03]

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

[commit-bot: I haz the power, 2017-06-01 16:42:25]

Dry run: CQ is trying da patch.

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

[James Cook, 2017-06-01 16:45:56]

jamescook@chromium.org changed reviewers:
+ azurewei@chromium.org

[James Cook, 2017-06-01 16:45:57]

azurewei, please take a look.

[commit-bot: I haz the power, 2017-06-01 17:44:43]

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

[commit-bot: I haz the power, 2017-06-01 17:44:44]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[James Cook, 2017-06-01 22:49:19]

+cc Hadi just FYI

[James Cook, 2017-06-02 16:27:22]

azurewei, please don't review this yet. I might need to take a different
approach.

[Azure Wei, 2017-06-05 06:29:47]

On 2017/06/02 16:27:22, James Cook wrote:
> azurewei, please don't review this yet. I might need to take a different
> approach.

OK. Thanks for the update.

[James Cook, 2017-06-05 15:35:11]

Abandoned. I'm going to take a different approach, using mojo.