Fix some flaky IME tests caused by incorrect selection update

Fix some flaky IME tests caused by incorrect selection update

Some IME tests are flaky because of wrong selection update. We suspect
it's because sometimes when we focus on input_text, the selection update
comes after reseting the state. We should wait to ensure the correct order.

BUG=628964
Committed: https://crrev.com/6a72722a3b04df4326bbf805676fca794fe33906
Cr-Commit-Position: refs/heads/master@{#440381}

[yabinh, 2016-12-22 09:33:17]

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

[commit-bot: I haz the power, 2016-12-22 09:33:31]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yabinh, 2016-12-22 09:35:46]

yabinh@chromium.org changed reviewers:
+ aelias@chromium.org, changwan@chromium.org

[yabinh, 2016-12-22 09:35:46]

PTAL, thanks!

[commit-bot: I haz the power, 2016-12-22 09:55:47]

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

[commit-bot: I haz the power, 2016-12-22 09:55:48]

Dry run: This issue passed the CQ dry run.

[Changwan Ryu, 2016-12-22 10:01:39]

lgtm, thanks!

https://codereview.chromium.org/2600453002/diff/1/content/public/android/java...
File
content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java
(right):

https://codereview.chromium.org/2600453002/diff/1/content/public/android/java...
content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java:97:
waitAndVerifyUpdateSelection(0, 0, 0, -1, -1);
RenderWidget::resetInputMethod() indeed calls UpdateTextInputState() *after*
initiating restart input. But normally it should be ignored unless there was
some change, in text_input_info_ or can_compose_inline_. In any case, redundant
state update should do no harm, so I agree that fixing the test should be enough
for now.

[yabinh, 2016-12-22 11:18:25]

The CQ bit was checked by yabinh@chromium.org

[commit-bot: I haz the power, 2016-12-22 11:18:40]

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, 2016-12-22 11:22:06]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482405505374130, "parent_rev":
"a9224087bf8cc4cbbd7fba4b66860e6fe36609c8", "commit_rev":
"7f80c288b820d35aab125e1509cefbb8fc651d93"}

[commit-bot: I haz the power, 2016-12-22 11:22:34]

Description was changed from

==========
Fix some flaky IME tests caused by incorrect selection update

Some IME tests are flaky because of wrong selection update. We suspect
it's because sometimes when we focus on input_text, the selection update
comes after reseting the state. We should wait to ensure the correct order.

BUG=628964
==========

to

==========
Fix some flaky IME tests caused by incorrect selection update

Some IME tests are flaky because of wrong selection update. We suspect
it's because sometimes when we focus on input_text, the selection update
comes after reseting the state. We should wait to ensure the correct order.

BUG=628964

Review-Url: https://codereview.chromium.org/2600453002
==========

[commit-bot: I haz the power, 2016-12-22 11:22:35]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-22 11:25:11]

Description was changed from

==========
Fix some flaky IME tests caused by incorrect selection update

Some IME tests are flaky because of wrong selection update. We suspect
it's because sometimes when we focus on input_text, the selection update
comes after reseting the state. We should wait to ensure the correct order.

BUG=628964

Review-Url: https://codereview.chromium.org/2600453002
==========

to

==========
Fix some flaky IME tests caused by incorrect selection update

Some IME tests are flaky because of wrong selection update. We suspect
it's because sometimes when we focus on input_text, the selection update
comes after reseting the state. We should wait to ensure the correct order.

BUG=628964

Committed: https://crrev.com/6a72722a3b04df4326bbf805676fca794fe33906
Cr-Commit-Position: refs/heads/master@{#440381}
==========

[commit-bot: I haz the power, 2016-12-22 11:25:12]

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