Add autotests to check that the IME works.

Add autotests to check that the IME works.
- Check that ibus-daemon launches when an IME is added.
- Check that ibus-daemon is killed when all IMEs are removed.
- Check that you can enable and enter text in mozc, chewing, hangul and pinyin

Change-Id: I389affd635685ef8387fad713fe63d8d45ee49df

BUG=chromium-os:4639
TEST=Run the desktopui_ImeTest autotest

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=74144b2

[Yusuke Sato, 2010-09-17 08:08:09]

LGTM

http://codereview.chromium.org/3391007/diff/7001/8002
File client/site_tests/desktopui_ImeTest/desktopui_ImeTest.py (right):

http://codereview.chromium.org/3391007/diff/7001/8002#newcode48
client/site_tests/desktopui_ImeTest/desktopui_ImeTest.py:48: time.sleep(3)
can you add a comment to explain why sleep() is necessary here?

http://codereview.chromium.org/3391007/diff/7001/8002#newcode67
client/site_tests/desktopui_ImeTest/desktopui_ImeTest.py:67: time.sleep(1)
ditto.

[ericli, 2010-09-18 22:08:32]

Any chance you could help to add this into nightly control suite? Thanks.

Eric

On Fri, Sep 17, 2010 at 1:08 AM, <yusukes@chromium.org> wrote:

> LGTM
>
>
> http://codereview.chromium.org/3391007/diff/7001/8002
> File client/site_tests/desktopui_ImeTest/desktopui_ImeTest.py (right):
>
> http://codereview.chromium.org/3391007/diff/7001/8002#newcode48
> client/site_tests/desktopui_ImeTest/desktopui_ImeTest.py:48:
> time.sleep(3)
> can you add a comment to explain why sleep() is necessary here?
>
> http://codereview.chromium.org/3391007/diff/7001/8002#newcode67
> client/site_tests/desktopui_ImeTest/desktopui_ImeTest.py:67:
> time.sleep(1)
> ditto.
>
> http://codereview.chromium.org/3391007/show
>



-- 
Eric Li
李咏竹
Google Kirkland

[Zachary Kuznia, 2010-09-21 01:11:20]

Certainly!  Where would I find the docs on this?


On 2010/09/18 22:08:32, ericli wrote:
> Any chance you could help to add this into nightly control suite? Thanks.
> 
> Eric
> 
> On Fri, Sep 17, 2010 at 1:08 AM, <mailto:yusukes@chromium.org> wrote:
> 
> > LGTM
> >
> >
> > http://codereview.chromium.org/3391007/diff/7001/8002
> > File client/site_tests/desktopui_ImeTest/desktopui_ImeTest.py (right):
> >
> > http://codereview.chromium.org/3391007/diff/7001/8002#newcode48
> > client/site_tests/desktopui_ImeTest/desktopui_ImeTest.py:48:
> > time.sleep(3)
> > can you add a comment to explain why sleep() is necessary here?
> >
> > http://codereview.chromium.org/3391007/diff/7001/8002#newcode67
> > client/site_tests/desktopui_ImeTest/desktopui_ImeTest.py:67:
> > time.sleep(1)
> > ditto.
> >
> > http://codereview.chromium.org/3391007/show
> >
> 
> 
> 
> -- 
> Eric Li
> 李咏竹
> Google Kirkland
>

[Zachary Kuznia, 2010-09-21 01:17:27]

http://codereview.chromium.org/3391007/diff/7001/8002
File client/site_tests/desktopui_ImeTest/desktopui_ImeTest.py (right):

http://codereview.chromium.org/3391007/diff/7001/8002#newcode48
client/site_tests/desktopui_ImeTest/desktopui_ImeTest.py:48: time.sleep(3)
On 2010/09/17 08:08:14, Yusuke Sato wrote:
> can you add a comment to explain why sleep() is necessary here?

Done.

http://codereview.chromium.org/3391007/diff/7001/8002#newcode67
client/site_tests/desktopui_ImeTest/desktopui_ImeTest.py:67: time.sleep(1)
On 2010/09/17 08:08:14, Yusuke Sato wrote:
> ditto.

Done.