Bug 33460

Summary: REGRESSION(53060): http/tests/security/xssAuditor/malformed-HTML.html timed out on Qt Linux Release Bot
Product: WebKit Reporter: Eric Seidel (no email) <eric>
Component: Tools / TestsAssignee: QtWebKit Unassigned <webkit-qt-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, ademar, commit-queue, dbates, hausmann, ossy, robert
Priority: P2 Keywords: Qt
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 33297    
Attachments:
Description Flags
Enable the test again none

Description Eric Seidel (no email) 2010-01-10 20:30:09 PST
http/tests/security/xssAuditor/malformed-HTML.html timed out on Qt Linux Release Bot

http://build.webkit.org/results/Qt%20Linux%20Release/r53060%20(5811)/results.html

There is no timeout message, so I assume this was a perl-script triggered timeout, not a DRT timeout.

http://trac.webkit.org/browser/trunk/LayoutTests/http/tests/security/xssAuditor/malformed-HTML.html
Comment 1 Eric Seidel (no email) 2010-01-10 21:19:40 PST
Strange.  This seems to be the failure of the night.  It's failed 3 times tonight.  I'd never noticed it before today.  The latest failure:
http://build.webkit.org/results/Qt%20Linux%20Release/r53062%20(5813)/results.html
Comment 2 Eric Seidel (no email) 2010-01-10 21:20:58 PST
Never mind.  This is just a regression from r53060.

Since the commit-queue landed r53060, I'll go ahead and roll it out.
Comment 3 Eric Seidel (no email) 2010-01-10 21:21:39 PST
http://trac.webkit.org/changeset/53060  Original bug was bug 33419.
Comment 4 Daniel Bates 2010-01-10 21:22:19 PST
Can we just skip it instead?
Comment 5 Eric Seidel (no email) 2010-01-10 21:24:15 PST
If that's what you'd prefer, that's fine. You have my rubber-stamp.  You're certainly much more involved with the Qt port than I am.
Comment 6 Eric Seidel (no email) 2010-01-10 21:24:43 PST
We just can't leave the bots red is all. :)
Comment 7 Daniel Bates 2010-01-10 21:37:42 PST
Added test to Qt Skipped file <http://trac.webkit.org/changeset/53064>.
Comment 8 Ademar Reis 2010-08-17 15:13:52 PDT
Created attachment 64645 [details]
Enable the test again

Tested on different scenarios and machines with qt-4.7 and qt-4.6, never failed.

I believe it's safe to enable it again.
Comment 9 Daniel Bates 2010-08-17 15:59:14 PDT
Comment on attachment 64645 [details]
Enable the test again

Sure, let's try this again.
Comment 10 WebKit Commit Bot 2010-08-17 18:56:06 PDT
Comment on attachment 64645 [details]
Enable the test again

Clearing flags on attachment: 64645

Committed r65584: <http://trac.webkit.org/changeset/65584>
Comment 11 WebKit Commit Bot 2010-08-17 18:56:12 PDT
All reviewed patches have been landed.  Closing bug.