New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 646231 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

"org.chromium.content.browser.input.ImeTest#testImeStaysOnLongPressingDifferentNonEmptyInputs" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Sep 13 2016

Issue description

"org.chromium.content.browser.input.ImeTest#testImeStaysOnLongPressingDifferentNonEmptyInputs" is flaky.

This issue was created automatically by the chromium-try-flakes app. Please find the right owner to fix the respective test/step and assign this issue to them. If the step/test is infrastructure-related, please add Infra-Troopers label and change issue status to Untriaged. When done, please remove the issue from Sheriff Bug Queue by removing the Sheriff-Chromium label.

We have detected 3 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyZwsSBUZsYWtlIlxvcmcuY2hyb21pdW0uY29udGVudC5icm93c2VyLmlucHV0LkltZVRlc3QjdGVzdEltZVN0YXlzT25Mb25nUHJlc3NpbmdEaWZmZXJlbnROb25FbXB0eUlucHV0cww.

Flaky tests should be disabled within 30 minutes unless culprit CL is found and reverted. Please see more details here: https://sites.google.com/a/chromium.org/dev/developers/tree-sheriffs/sheriffing-bug-queues#triaging-auto-filed-flakiness-bugs
 
Cc: aelias@chromium.org changwan@chromium.org
Appears to be significantly flakier on M than before: http://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=content_shell_test_apk&tests=org.chromium.content.browser.input.ImeTest%23testImeStaysOnLongPressingDifferentNonEmptyInputs

+cc aelias, changwan as the closest OWNERS
Components: UI>Input>Text>IME
Labels: -Sheriff-Chromium OS-Android
I'm disabling this flaky test in https://codereview.chromium.org/2338743002/
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 13 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/349a84be7fd236b06e70eb5b0c412fbdeed384b3

commit 349a84be7fd236b06e70eb5b0c412fbdeed384b3
Author: kjellander <kjellander@chromium.org>
Date: Tue Sep 13 09:17:27 2016

Disable org.chromium.content.browser.input.ImeTest#testImeStaysOnLongPressingDifferentNonEmptyInputs

This test is flaky on Android.

BUG= 646231 
TBR=aelias@chromium.org

Review-Url: https://codereview.chromium.org/2338743002
Cr-Commit-Position: refs/heads/master@{#418202}

[modify] https://crrev.com/349a84be7fd236b06e70eb5b0c412fbdeed384b3/content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java

Comment 4 by yabinh@chromium.org, Sep 20 2016

Owner: yabinh@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 5 by bugdroid1@chromium.org, Sep 20 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f1522380549d64191081ef505466c43d32816b68

commit f1522380549d64191081ef505466c43d32816b68
Author: yabinh <yabinh@chromium.org>
Date: Tue Sep 20 10:12:29 2016

Fix flaky ImeTest#testImeStaysOnLongPressingDifferentNonEmptyInputs

commitText() runs on ImeThread and focusNode() runs on UIThread, which
causes a race condition. We should make sure that focusNode() runs after
commitText() finished.

BUG= 628964 ,  646231 

Review-Url: https://codereview.chromium.org/2356723002
Cr-Commit-Position: refs/heads/master@{#419714}

[modify] https://crrev.com/f1522380549d64191081ef505466c43d32816b68/content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java

Comment 6 by yabinh@chromium.org, Sep 26 2016

Status: Fixed (was: Assigned)

Sign in to add a comment