Bug 88131

Summary:

Layout Test fast/dom/HTMLMeterElement/meter-element-crash.html is failing

Product:

WebKit

Reporter:

Stephen Chenney <schenney>

Component:

Tools / Tests

Assignee:

James Robinson <jamesr>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ap, dglazkov, jamesr, jonlee, morrita, rniwa

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	tony: review+

Description Stephen Chenney 2012-06-01 13:42:12 PDT

The following layout test is failing on Mac and Win debug:

fast/dom/HTMLMeterElement/meter-element-crash.html

This test has been flaky before, see http://trac.webkit.org/changeset/94634

Comment 1 Stephen Chenney 2012-06-01 13:47:17 PDT

Committed r119277: <http://trac.webkit.org/changeset/119277>

Comment 2 Stephen Chenney 2012-06-01 13:47:40 PDT

Committed expectations update.

Comment 3 Alexey Proskuryakov 2012-06-01 13:50:02 PDT

Do you have a stack trace to post?

Comment 4 Ryosuke Niwa 2012-06-01 13:51:34 PDT

(In reply to comment #3)
> Do you have a stack trace to post?

Unfortunately Chromium bots delete symbols in WebKit/WebCore so we can't see anything useful. We should really change that though... it makes impossible to diagnose the problems :(

Comment 5 Hajime Morrita 2012-06-03 21:37:31 PDT

Was this crashing? If it was just failing.
I could make test result more stable somehow.

Comment 6 James Robinson 2013-03-08 13:08:54 PST

Created attachment 192277 [details]
Patch

Comment 7 James Robinson 2013-03-08 13:09:32 PST

I believe this patch will de-flake the test.  Without this patch, locally (cr linux debug) the test fails 6-20 times when run with --repeat-each=10 --iterations=10.  With this patch is passes 100/100 times.

Comment 8 Tony Chang 2013-03-08 13:12:27 PST

Comment on attachment 192277 [details]
Patch

Should there be a TestExpectations change to re-enable this test?

Comment 9 James Robinson 2013-03-08 13:14:24 PST

(In reply to comment #8)
> (From update of attachment 192277 [details])
> Should there be a TestExpectations change to re-enable this test?

I accidentally landed that part already in: http://trac.webkit.org/changeset/145259/trunk/LayoutTests/platform/chromium/TestExpectations

Comment 10 James Robinson 2013-03-08 13:22:24 PST

Committed r145263: <http://trac.webkit.org/changeset/145263>