New issue
Advanced search Search tips

Issue 779849 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

AccessibilityBoundsInherits is flaky

Project Member Reported by katydek@google.com, Oct 30 2017

Issue description

https://findit-for-me.appspot.com/waterfall/flake/flake-culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyQwsSDEZsYWtlQ3VscHJpdCIxY2hyb21pdW0vNDA0MjBhMzg5MWJlNzNiNjBmNzY1Y2I3ZTc3MmU5MDFkZThkMmVkMAw

Found in https://logs.chromium.org/v/?s=chromium%2Fbb%2Fchromium.fyi%2FBrowser_Side_Navigation_Linux%2F33946%2F%2B%2Frecipes%2Fsteps%2Fcontent_browsertests%2F0%2Fstdout:

[ RUN      ] DumpAccessibilityTreeTest.AccessibilityBoundsInherits
Xlib:  extension "RANDR" missing on display ":99".
DevTools listening on ws://127.0.0.1:37374/devtools/browser/091f2e3a-8889-491a-ba30-47775d11a679
[14745:14745:1025/234748.540203:38065016163:INFO:dump_accessibility_browsertest_base.cc(240)] Testing: /b/s/w/ir/content/test/data/accessibility/html/bounds-inherits.html
[14745:14745:1025/234748.540251:38065016207:INFO:dump_accessibility_browsertest_base.cc(242)] Expected output: /b/s/w/ir/content/test/data/accessibility/html/bounds-inherits-expected-blink.txt
../../content/browser/accessibility/dump_accessibility_browsertest_base.cc:371: Failure
Value of: is_different
  Actual: true
Expected: false
[14745:14745:1025/234748.571547:38065047506:ERROR:dump_accessibility_browsertest_base.cc(396)] Diff:
* Line Expected
- ---- --------
*    1 rootWebArea size=(800, 600)
     2 ++genericContainer size=(200, 200)
     3 ++++genericContainer offscreen size=(0, 0)
     4 ++++++genericContainer offscreen size=(0, 0)
     5 <-- End-of-file -->
     6 <-- End-of-file -->
Actual
------
rootWebArea size=(796, 560)
++genericContainer size=(200, 200)
++++genericContainer offscreen size=(0, 0)
++++++genericContainer offscreen size=(0, 0)
<-- End-of-file -->
 

Comment 1 by katydek@google.com, Oct 30 2017

Cc: dmazz...@chromium.org

Comment 2 by katydek@google.com, Oct 31 2017

Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Oct 31 2017

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

commit c58c2da70f31fec81bfad06ad885a1a6ca6b95ac
Author: Katie Dektar <katydek@google.com>
Date: Tue Oct 31 15:59:59 2017

Fix flaky test by removing root view size, which could vary.

Bug:  779849 
Change-Id: I0a3ad826fbfa2390afa8fbc26e6f6ebaa24e77a6
Reviewed-on: https://chromium-review.googlesource.com/745309
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#512841}
[modify] https://crrev.com/c58c2da70f31fec81bfad06ad885a1a6ca6b95ac/content/test/data/accessibility/html/bounds-inherits-expected-blink.txt
[modify] https://crrev.com/c58c2da70f31fec81bfad06ad885a1a6ca6b95ac/content/test/data/accessibility/html/bounds-inherits.html

Comment 4 by katydek@google.com, Oct 31 2017

Status: Fixed (was: Started)

Sign in to add a comment