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

Issue 778515 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Nov 2017
EstimatedDays: ----
NextAction: ----
OS: Android , Windows , Mac
Pri: ----
Type: ----



Sign in to add a comment

webkit_unit_tests failing on chromium.webkit/WebKit Android (Nexus4)

Project Member Reported by peria@chromium.org, Oct 26 2017

Issue description

webkit_unit_tests failing on chromium.webkit/WebKit Android (Nexus4)

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


After landing https://chromium-review.googlesource.com/736868 , six tests of transformed tests fail.

Unexpected Failures:
* http/tests/devtools/elements/styles-3/styles-add-new-rule-tab.js
* http/tests/devtools/elements/styles-3/styles-add-new-rule.js
* http/tests/devtools/elements/styles-3/styles-disable-inherited.js
* virtual/mojo-loading/http/tests/devtools/elements/styles-3/styles-add-new-rule-tab.js
* virtual/mojo-loading/http/tests/devtools/elements/styles-3/styles-add-new-rule.js
* virtual/mojo-loading/http/tests/devtools/elements/styles-3/styles-disable-inherited.js
 
Labels: OS-Windows
styles-add-new-rule-tab.js fails on Win Dbg consistently https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win7%20%28dbg%29/builds/11762

The diff is
--- e:\b\s\w\ioh6owi6\layout-test-results\retry_1\virtual/mojo-loading/http/tests/devtools/elements/styles-3/styles-add-new-rule-tab-expected.txt
+++ e:\b\s\w\ioh6owi6\layout-test-results\retry_1\virtual/mojo-loading/http/tests/devtools/elements/styles-3/styles-add-new-rule-tab-actual.txt
@@ -7,7 +7,6 @@
 
 [expanded] 
 foo, [$div#inspected, $]bar { (inspector-stylesheet:1 -> inspector-stylesheet:1:26)
-    color: maroon;
 
 [expanded] 
 [$div$] { (user agent stylesheet)

That is "color: maroon;" is missing. Looks like a bug.
The same about http/tests/devtools/elements/styles-3/styles-add-new-rule.js
"color: maroon;" is missing.
Project Member

Comment 4 by bugdroid1@chromium.org, Oct 26 2017

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

commit d991b11eb365321e190ed016e041fe5ef9a2c02a
Author: Vasilii Sukhanov <vasilii@chromium.org>
Date: Thu Oct 26 12:50:32 2017

Update expectations for devtools/elements/styles-3 tests

virtual/mojo-loading/http/tests/devtools/elements/styles-3/styles-add-new-rule-tab.js and http/tests/devtools/elements/styles-3/styles-add-new-rule.js fail on Win Dbg.

TBR=chenwilliam@chromium.org

Bug:  778515 
Change-Id: I6ac3bccd6f465dedaacb55ffcc8452d966d3ded7
Reviewed-on: https://chromium-review.googlesource.com/738246
Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#511798}
[modify] https://crrev.com/d991b11eb365321e190ed016e041fe5ef9a2c02a/third_party/WebKit/LayoutTests/TestExpectations

Comment 6 by meade@chromium.org, Nov 6 2017

Labels: -Sheriff-Chromium
Status: Fixed (was: Available)
The 3 failing tests mentioned in #1 are no longer failing. Removing sheriff label.

Sign in to add a comment