Bug 32470 - [Qt] Missing forward entries when dumping back/forward list
Summary: [Qt] Missing forward entries when dumping back/forward list
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P1 Normal
Assignee: QtWebKit Unassigned
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks:
 
Reported: 2009-12-12 02:07 PST by Csaba Osztrogonác
Modified: 2010-06-21 14:08 PDT (History)
4 users (show)

See Also:

Attachments
Patch (2.10 KB, patch)
2010-06-20 12:06 PDT, Robert Hogan 		kenneth: review+
kenneth: commit-queue-
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 Csaba Osztrogonác 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.
Comment 1 Csaba Osztrogonác 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
Comment 2 Csaba Osztrogonác 2009-12-12 02:15:53 PST 
original bug: https://bugs.webkit.org/show_bug.cgi?id=32383
Comment 3 Jocelyn Turcotte 2010-03-16 12:48:42 PDT 
Setting as P1 since its a regression.
Comment 4 Robert Hogan 2010-06-20 12:06:14 PDT 
Created attachment 59207 [details]
Patch
Comment 5 Csaba Osztrogonác 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. ;)
Comment 6 Kenneth Rohde Christiansen 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
Comment 7 Robert Hogan 2010-06-21 14:08:11 PDT 
Committed r61578: <http://trac.webkit.org/changeset/61578>