New issue
Advanced search Search tips

Issue 731825 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug
Team-Accessibility



Sign in to add a comment

"EditingTest.RichTextMoveByCharacterNodeWorkaround" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Jun 9 2017

Issue description

"EditingTest.RichTextMoveByCharacterNodeWorkaround" 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 4 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyPAsSBUZsYWtlIjFFZGl0aW5nVGVzdC5SaWNoVGV4dE1vdmVCeUNoYXJhY3Rlck5vZGVXb3JrYXJvdW5kDA.

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
 
Project Member

Comment 1 by chromium...@appspot.gserviceaccount.com, Jun 12 2017

Detected 3 new flakes for test/step "EditingTest.RichTextMoveByCharacterNodeWorkaround". To see the actual flakes, please visit https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyPAsSBUZsYWtlIjFFZGl0aW5nVGVzdC5SaWNoVGV4dE1vdmVCeUNoYXJhY3Rlck5vZGVXb3JrYXJvdW5kDA. This message was posted automatically by the chromium-try-flakes app.
Owner: dtseng@chromium.org
Hi dtseng@, is this possible another flaky test you can triage in EditingTest?
Status: Assigned (was: Untriaged)
There are 3 flaky tests in RichTextMove***, this,  issue 732000 , and  issue 724847 . All should be disabled.
Didn't hear from author so submitting https://chromium-review.googlesource.com/c/533594/
Components: UI>Accessibility>ChromeVox
Labels: -Pri-1 Pri-3
Disable in flight, dropping to p3.
Project Member

Comment 7 by chromium...@appspot.gserviceaccount.com, Jun 13 2017

Detected 6 new flakes for test/step "EditingTest.RichTextMoveByCharacterNodeWorkaround". To see the actual flakes, please visit https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyPAsSBUZsYWtlIjFFZGl0aW5nVGVzdC5SaWNoVGV4dE1vdmVCeUNoYXJhY3Rlck5vZGVXb3JrYXJvdW5kDA. This message was posted automatically by the chromium-try-flakes app.
Labels: -Sheriff-Chromium
Removing sheriff tracking since these have workarounds in place.
Project Member

Comment 9 by bugdroid1@chromium.org, Jun 13 2017

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

commit e68adf216466ddb631e88c39987be66435c69fce
Author: Dale Curtis <dalecurtis@chromium.org>
Date: Tue Jun 13 18:58:37 2017

Disabled flaky failing EditingTest.RichTextMove* tests.

BUG= 731825 , 732000 , 724847 
TEST=none
TBR=dtseng

Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I3428fcfc7a45b95bc1a38059b28610c212b588e0
Reviewed-on: https://chromium-review.googlesource.com/533594
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#479084}
[modify] https://crrev.com/e68adf216466ddb631e88c39987be66435c69fce/chrome/browser/resources/chromeos/chromevox/cvox2/background/editing_test.extjs

Project Member

Comment 10 by bugdroid1@chromium.org, Jun 14 2017

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

commit 7b2d9fd6510ad34f7530a616586a93c68eb0b10e
Author: dtseng <dtseng@chromium.org>
Date: Wed Jun 14 04:19:17 2017

Fix EditingTest.RichText*

BUG= 724847 , 731825 , 732000 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/7b2d9fd6510ad34f7530a616586a93c68eb0b10e/chrome/browser/resources/chromeos/chromevox/cvox2/background/editing_test.extjs

Status: fixed (was: Assigned)

Sign in to add a comment