New issue
Advanced search Search tips

Issue 714097 link

Starred by 0 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Jan 7
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug



Sign in to add a comment

nav-two-elements-one-line.html fails on chromium.webkit/WebKit Mac10.11 (dbg)

Project Member Reported by fhorschig@chromium.org, Apr 21 2017

Issue description

Builders failed on: 
- WebKit Mac10.11 (dbg): 
  https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.11%20%28dbg%29


Diff between expected and actual two-elements-one-line.html:

--- /b/rr/tmpjcG2P9/w/layout-test-results/fast/spatial-navigation/snav-two-elements-one-line-expected.txt
+++ /b/rr/tmpjcG2P9/w/layout-test-results/fast/spatial-navigation/snav-two-elements-one-line-actual.txt
@@ -5,10 +5,10 @@
 PASS successfullyParsed is true
 
 TEST COMPLETE
-PASS gFocusedDocument.activeElement.getAttribute("id") is "e1"
-PASS gFocusedDocument.activeElement.getAttribute("id") is "e2"
-PASS gFocusedDocument.activeElement.getAttribute("id") is "e3"
-PASS gFocusedDocument.activeElement.getAttribute("id") is "e2"
-PASS gFocusedDocument.activeElement.getAttribute("id") is "e1"
+FAIL gFocusedDocument.activeElement.getAttribute("id") should be e1. Was start.
+FAIL gFocusedDocument.activeElement.getAttribute("id") should be e2. Was e1.
+FAIL gFocusedDocument.activeElement.getAttribute("id") should be e3. Was e2.
+FAIL gFocusedDocument.activeElement.getAttribute("id") should be e2. Was e1.
+FAIL gFocusedDocument.activeElement.getAttribute("id") should be e1. Was start.
 PASS gFocusedDocument.activeElement.getAttribute("id") is "start"
 This test is testing that we can handle 2 inline elements in the same line.


Most likely causing the failure (CC'ed):

58cb30d4 (varkha): MacViews: Allows the toolkit-views Manage Passwords Dialog to be used
74934b2b (jzfeng): use devtools command to do window management


Full regression range:

2d9f45d2 (rlanday): Rewrite loops in DocumentMarkerController to use ListForType()
5e87893e (vasilii): Notify PasswordAutofillAgent about dynamic forms after the document was parsed.
39a09b97 (yoichio): Remove LayoutView from LayoutSelection::commit()
019c92b5 (robhogan): Treat cells with percent height as auto if their container depends on content height
7fd3f671 (ollel): Add curly brackets to list of characters that gn needs to escape
58cb30d4 (varkha): MacViews: Allows the toolkit-views Manage Passwords Dialog to be used
74934b2b (jzfeng): use devtools command to do window management
3a05731e (gambard): LabelObserver is no longer retained by the label
7d42d2d5 (vasilii): Enable PasswordManagerBrowserTestBase.PreferPasswordFormManagerWhichFinishedMatching.


 
Labels: -Pri-2 Pri-3
Either the test has been fixed or it's flaky. Leaving this bug open for further sample collection.

Successful succeeding build:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.11%20%28dbg%29/builds/8435

Comment 2 by bokan@chromium.org, Apr 21 2017

Components: -Blink Blink>Focus
Labels: -Sheriff-Chromium
Owner: fhorschig@chromium.org
Status: Assigned (was: Untriaged)
The last 22 builds have succeeded:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.11%20%28dbg%29?numbuilds=25

Assigning to Friedrich since he wants to keep an eye on it.

Friedrich: if you no longer want to keep an eye on this can you find a better owner?  Thanks.

Owner: ----
Status: Fixed (was: Assigned)
I am neither Blink member nor Sheriff anymore. So no, I don't want to keep an eye on Mac Webkit tests ;)

Marking this bug as fixed for now.
Components: Blink>HTML>Focus
Components: -Blink>Focus
Labels: OS-Mac
Status: Available (was: Fixed)
This test is failing again:
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Mac10.11%20%28dbg%29/builds/11764

Diff:
--- /b/s/w/ioQcYxj6/layout-test-results/fast/spatial-navigation/snav-two-elements-one-line-expected.txt
+++ /b/s/w/ioQcYxj6/layout-test-results/fast/spatial-navigation/snav-two-elements-one-line-actual.txt
@@ -5,10 +5,10 @@
 PASS successfullyParsed is true
 
 TEST COMPLETE
-PASS gFocusedDocument.activeElement.getAttribute("id") is "e1"
-PASS gFocusedDocument.activeElement.getAttribute("id") is "e2"
-PASS gFocusedDocument.activeElement.getAttribute("id") is "e3"
-PASS gFocusedDocument.activeElement.getAttribute("id") is "e2"
-PASS gFocusedDocument.activeElement.getAttribute("id") is "e1"
+FAIL gFocusedDocument.activeElement.getAttribute("id") should be e1. Was start.
+FAIL gFocusedDocument.activeElement.getAttribute("id") should be e2. Was e1.
+FAIL gFocusedDocument.activeElement.getAttribute("id") should be e3. Was e2.
+FAIL gFocusedDocument.activeElement.getAttribute("id") should be e2. Was e1.
+FAIL gFocusedDocument.activeElement.getAttribute("id") should be e1. Was start.
 PASS gFocusedDocument.activeElement.getAttribute("id") is "start"
 This test is testing that we can handle 2 inline elements in the same line.
Project Member

Comment 8 by sheriffbot@chromium.org, Jan 3

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Hotlist-Recharge-Cold Test-WebTest
Status: WontFix (was: Untriaged)
Summary: nav-two-elements-one-line.html fails on chromium.webkit/WebKit Mac10.11 (dbg) (was: webkit_tests failing on chromium.webkit/WebKit Mac10.11 (dbg))
Looks the test is very stable now.


Sign in to add a comment