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

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>