New issue
Advanced search Search tips

Issue 664274 link

Starred by 2 users
Status: WontFix
Owner: ----
Closed: Jan 2018
Cc:
qyears...@chromium.org
jeffcarp@chromium.org
dpranke@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Layout test results.html shows pass/pass results under "failures"

Project Member Reported by qyears...@chromium.org, Nov 10 2016 
Example:

1. Go to https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Precise/2241/layout-test-results/results.html

2. Show "expected failures"

This lists lots of tests that are expected pass, actual pass under "failures", but it should only show failures.

Comment 1 by qyears...@chromium.org, Dec 9 2016

Status: Available (was: Unconfirmed)
Specifically, what's wrong here is:

In results.html, the function processGlobalStateFor(testObject) puts tests into buckets. The default bucket is called "failingTests" and there's no "passingTests" bucket.

Most obvious fix: Add a passingTests bucket and either display that in a separate list with a separate check-box, or just don't display it, or just don't keep track of passing tests.

See: https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/fast/harness/results.html?l=471

Comment 2 by qyears...@chromium.org, May 30 2017

Labels: Hotlist-GoodFirstBug
T

Comment 3 by qyears...@chromium.org, Aug 24 2017 

 Issue 758393  has been merged into this issue.

Comment 4 by kereliuk@chromium.org, Jan 19 2018

Owner: qyears...@chromium.org
@qyearsley assigning you to be the owner for now

Comment 5 by qyears...@chromium.org, Jan 19 2018

Labels: -Pri-2 -Hotlist-GoodFirstBug Pri-3
Owner: ----
Status: WontFix (was: Available)
Marking as WontFix since we have now switched to a different layout test results viewer by default.
►

Sign in to add a comment