Chromium Code Reviews
DescriptionRevert of Remove all the @FlakyTest annotations in ImeTest (patchset #2 id:20001 of https://codereview.chromium.org/2058633004/ )
Reason for revert:
Flaky failure seen in
https://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg%29/builds/34878
Original issue's description:
> There are 6 flaky tests in ImeTest. They are flaky because when we call
> InputConnection#setSelection and other methods together, the order of messages
> sometimes does not get kept.
>
> We can solve this by enabling IME thread, because when IME thead is on,
> the order can be kept.
>
> Another way is applying the following patch, which deals with the re-ordering
> issue: https://codereview.chromium.org/1877073003/. Since the patch has been
> landed, we can remove the @FlakyTest annotations.
>
> BUG=598482, 603991
>
> Committed: https://crrev.com/6f23392b1f1f2b91909f646bc5ddf54c2c4003a6
> Cr-Commit-Position: refs/heads/master@{#400099}
TBR=changwan@chromium.org,yabinh@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=598482, 603991
Committed: https://crrev.com/186c15876ee97b4d0be6302c0635814c873448f5
Cr-Commit-Position: refs/heads/master@{#400151}
Patch Set 1 #
Messages
Total messages: 8 (3 generated)
|
|||||||||||||||||||