Bug 32470

Summary: [Qt] Missing forward entries when dumping back/forward list
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: Tools / TestsAssignee: QtWebKit Unassigned <webkit-qt-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abecsi, jturcotte, kenneth, robert
Priority: P1 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Attachments:
Description Flags
Patch kenneth: review+, kenneth: commit-queue-

Csaba Osztrogonác
Reported 2009-12-12 02:07:54 PST
Two tests introduced in http://trac.webkit.org/changeset/52033 fail, necessary forward entries are missing. It needs investigation.
Attachments
Patch (2.10 KB, patch)
2010-06-20 12:06 PDT, Robert Hogan 		kenneth: review+
kenneth: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Csaba Osztrogonác
Comment 1 2009-12-12 02:14:22 PST
added to skiplist until fix by r52048: http/tests/navigation/dynamic-iframe-dynamic-form-back-entry.html http/tests/navigation/parsed-iframe-dynamic-form-back-entry.html
Csaba Osztrogonác
Comment 2 2009-12-12 02:15:53 PST
original bug: https://bugs.webkit.org/show_bug.cgi?id=32383
Jocelyn Turcotte
Comment 3 2010-03-16 12:48:42 PDT
Setting as P1 since its a regression.
Robert Hogan
Comment 4 2010-06-20 12:06:14 PDT
Created attachment 59207 [details] Patch
Csaba Osztrogonác
Comment 5 2010-06-20 15:29:27 PDT
(In reply to comment #4) > Created an attachment (id=59207) [details] > Patch Reallly. :) LGTM, but please fix duplicated ChangeLog. ;)
Kenneth Rohde Christiansen
Comment 6 2010-06-21 13:15:37 PDT
Comment on attachment 59207 [details] Patch You changelog somehow got there twise. r+, if you fix this when landing
Robert Hogan
Comment 7 2010-06-21 14:08:11 PDT
Committed r61578: <http://trac.webkit.org/changeset/61578>
Note You need to log in before you can comment on or make changes to this bug.