Bug 91112 - REGRESSION (r122450?): Layout Test fast/dom/attribute-empty-value-no-children.html is failing
Summary: REGRESSION (r122450?): Layout Test fast/dom/attribute-empty-value-no-children...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P1 Normal
Assignee: Andreas Kling
URL:
Keywords:
: 91254 (view as bug list)
Depends on:
Blocks:
 
Reported: 2012-07-12 09:52 PDT by W. James MacLean
Modified: 2012-07-30 04:03 PDT (History)
8 users (show)

See Also:

Attachments
Patch (1.54 KB, patch)
2012-07-13 07:51 PDT, Andreas Kling 		andersca: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description W. James MacLean 2012-07-12 09:52:42 PDT 
The following layout test is failing on Linux/Windows Debug

fast/dom/attribute-empty-value-no-children.html

Probable cause:

http://trac.webkit.org/changeset/122450
Comment 1 Andreas Kling 2012-07-13 05:29:43 PDT 
Hum. I can't reproduce this problem. Can I see this failure (diff/stacktrace/something else?) on the web somewhere?
Comment 2 W. James MacLean 2012-07-13 07:04:39 PDT 
(In reply to comment #1)
> Hum. I can't reproduce this problem. Can I see this failure (diff/stacktrace/something else?) on the web somewhere?

It's failing pretty consistently on the WebKit Linux (dbg) and WebKit Win (dbg)(1) test bots:

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=fast%2Fdom%2Fattribute-empty-value-no-children.html
Comment 3 Andreas Kling 2012-07-13 07:51:22 PDT 
Created attachment 152262 [details]
Patch
Comment 4 Andreas Kling 2012-07-13 08:59:01 PDT 
*** Bug 91254 has been marked as a duplicate of this bug. ***
Comment 5 Vsevolod Vlasov 2012-07-13 09:01:56 PDT 
Skipped: http://trac.webkit.org/changeset/122588

Please unskip before landing.
Comment 6 Andreas Kling 2012-07-30 04:03:38 PDT 
(In reply to comment #5)
> Skipped: http://trac.webkit.org/changeset/122588
> 
> Please unskip before landing.

It was unskipped already, and the patch that caused the regression was rolled out.