Show the IME notification if the system shelf is hidden

Show the IME notification if the system shelf is hidden

BUG=120576
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=149578

[Zachary Kuznia, 2012-08-01 09:24:59]

Hello sadrul,

Please review this at:
https://chromiumcodereview.appspot.com/10830114

Thanks,
-Zach

[sadrul, 2012-08-01 15:00:41]

http://codereview.chromium.org/10830114/diff/1/ash/system/ime/tray_ime.cc
File ash/system/ime/tray_ime.cc (right):

http://codereview.chromium.org/10830114/diff/1/ash/system/ime/tray_ime.cc#new...
ash/system/ime/tray_ime.cc:327: HideNotificationView();
Can we update the existing notification instead of recreating it? (e.g. we
update any existing detailed/default views instead of recreating them).

[Zachary Kuznia, 2012-08-02 01:51:18]

http://codereview.chromium.org/10830114/diff/1/ash/system/ime/tray_ime.cc
File ash/system/ime/tray_ime.cc (right):

http://codereview.chromium.org/10830114/diff/1/ash/system/ime/tray_ime.cc#new...
ash/system/ime/tray_ime.cc:327: HideNotificationView();
On 2012/08/01 15:00:41, sadrul wrote:
> Can we update the existing notification instead of recreating it? (e.g. we
> update any existing detailed/default views instead of recreating them).

Done.

[sadrul, 2012-08-02 02:52:10]

LGTM

[commit-bot: I haz the power, 2012-08-02 03:07:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/zork@chromium.org/10830114/3002

[commit-bot: I haz the power, 2012-08-02 03:23:30]

Try job failure for 10830114-3002 (retry) on android for steps "compile, build"
(clobber build).
It's a second try, previously, steps "compile, build" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android&nu...

[commit-bot: I haz the power, 2012-08-02 03:34:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/zork@chromium.org/10830114/3002

[commit-bot: I haz the power, 2012-08-02 05:27:06]

Change committed as 149578