Bug 41261

Summary: [Qt] dump frames in ascending alphabetical order of title
Product: WebKit Reporter: Robert Hogan <robert>
Component: WebKit Misc.Assignee: Nobody <webkit-unassigned>
Status: CLOSED FIXED    
Severity: Normal CC: abarth, ariya.hidayat, eric, webkit.review.bot
Priority: P2 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 41088    
Attachments:
Description Flags
Patch
none
Patch kenneth: review+

Robert Hogan
Reported 2010-06-27 10:22:03 PDT
Unskip: http/tests/navigation/metaredirect-frames.html http/tests/navigation/redirect302-frames.html http/tests/navigation/redirect302-subframeload.html http/tests/navigation/timerredirect-frames.html
Attachments
Patch (3.89 KB, patch)
2010-06-27 10:24 PDT, Robert Hogan 		no flags
DetailsFormatted DiffDiff
Patch (7.36 KB, patch)
2010-06-28 14:44 PDT, Robert Hogan 		kenneth: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Robert Hogan
Comment 1 2010-06-27 10:24:36 PDT
Created attachment 59857 [details] Patch
Robert Hogan
Comment 2 2010-06-28 12:07:57 PDT
Committed r62021: <http://trac.webkit.org/changeset/62021>
WebKit Review Bot
Comment 3 2010-06-28 12:28:54 PDT
http://trac.webkit.org/changeset/62021 might have broken Qt Linux Release
Robert Hogan
Comment 4 2010-06-28 12:38:56 PDT
Rolled out r62024.
Robert Hogan
Comment 5 2010-06-28 14:44:06 PDT
Created attachment 59944 [details] Patch I think I fell victim to the flakiness of these tests - my wrong-headed patch passed by a fluke when run locally. Apologies for thrashing the bots! This is the correct fix.
Ariya Hidayat
Comment 6 2010-08-11 10:21:57 PDT
Robert, do you want to land this patch?
Robert Hogan
Comment 7 2010-08-11 11:14:00 PDT
Whoops! Landed as r62296.
Note You need to log in before you can comment on or make changes to this bug.