New issue
Advanced search Search tips

Issue 911746 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 5
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----
Team-Accessibility



Sign in to add a comment

viz_browser_tests failing on chromium.chromiumos/linux-chromeos-rel

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Dec 4

Issue description

Project Member

Comment 1 by bugdroid1@chromium.org, Dec 4

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

commit 5c52cbb6f58874aac4f4b5369856af106c69fd47
Author: Ovidio Henriquez <odejesush@chromium.org>
Date: Tue Dec 04 19:30:17 2018

Revert "Add test for Switch Access Navigation Manager"

This reverts commit dc20b48bed57084acab94698ddef86a672182c1a.

Reason for revert: linux-chromeos-rel fails the following tests:
* SwitchAccessNavigationManagerTest.MoveBackward
* SwitchAccessNavigationManagerTest.MoveBackAndForth

Bug:  911746 

Original change's description:
> Add test for Switch Access Navigation Manager
> 
> Bug: 897365
> Change-Id: Ib0ff177788798706ec7f05111208f61c9946a7fb
> Reviewed-on: https://chromium-review.googlesource.com/c/1356002
> Commit-Queue: Anastasia Helfinstein <anastasi@google.com>
> Reviewed-by: David Tseng <dtseng@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#613607}

TBR=dtseng@chromium.org,anastasi@google.com

Change-Id: I476cc50da905be2fbfc9c75070e208575a3fa0eb
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 897365
Reviewed-on: https://chromium-review.googlesource.com/c/1361656
Reviewed-by: Ovidio Henriquez <odejesush@chromium.org>
Commit-Queue: Ovidio Henriquez <odejesush@chromium.org>
Cr-Commit-Position: refs/heads/master@{#613638}
[modify] https://crrev.com/5c52cbb6f58874aac4f4b5369856af106c69fd47/chrome/browser/resources/chromeos/switch_access/BUILD.gn
[delete] https://crrev.com/a0f2d87555e1357e9c43378e45ec0ebebda97232/chrome/browser/resources/chromeos/switch_access/navigation_manager_test.extjs

Components: UI>Accessibility
Status: Untriaged (was: Available)
Labels: -Sheriff-Chromium
Owner: anastasi@google.com
Status: Fixed (was: Untriaged)
This change was reverted here: https://chromium-review.googlesource.com/c/chromium/src/+/1361656.

This issue has been identified and the change is out for review again here: https://chromium-review.googlesource.com/c/chromium/src/+/1362465

Sign in to add a comment