Bug 74960

Summary: [Qt] [Gardening] Move Qt 4.8 tests to the right skip list.
Product: WebKit Reporter: João Paulo Rechi Vita <jprvita>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ossy, webkit.review.bot, zoltan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

João Paulo Rechi Vita
Reported 2011-12-20 15:00:27 PST
[Qt] [Gardening] Move WK1 tests to the right skip list.
Attachments
Patch (2.00 KB, patch)
2011-12-20 15:02 PST, João Paulo Rechi Vita 		no flags
DetailsFormatted DiffDiff
Patch (1.96 KB, patch)
2011-12-21 09:11 PST, João Paulo Rechi Vita 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
João Paulo Rechi Vita
Comment 1 2011-12-20 15:02:56 PST
Created attachment 120094 [details] Patch These tests pass on Qt-5.0+WK2 but fail on WK1. If this patch lands a comment should be written on https://bugs.webkit.org/show_bug.cgi?id=55767
Csaba Osztrogonác
Comment 2 2011-12-21 05:07:12 PST
Does this test fail with Qt 4.8 and 5.0 WK1 too? If yes, the bug and comment should be renamed to: [Qt] Two http auth tests fail with Qt >= 4.7.2
João Paulo Rechi Vita
Comment 3 2011-12-21 06:31:10 PST
(In reply to comment #2) > Does this test fail with Qt 4.8 and 5.0 WK1 too? If yes, the bug and comment should be renamed to: [Qt] Two http auth tests fail with Qt >= 4.7.2 I think I got a little confused here, sorry. This fails only on Qt 4.8, so it should actually be moved to the Qt 4.8 skip list. I'll fix this in a new patch.
João Paulo Rechi Vita
Comment 4 2011-12-21 09:11:29 PST
Created attachment 120183 [details] Patch
WebKit Review Bot
Comment 5 2011-12-22 01:28:33 PST
Comment on attachment 120183 [details] Patch Clearing flags on attachment: 120183 Committed r103504: <http://trac.webkit.org/changeset/103504>
WebKit Review Bot
Comment 6 2011-12-22 01:28:37 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.