Changeset 59660 in webkit

Timestamp:

May 18, 2010 1:42:28 AM (14 years ago)

Author:

Simon Hausmann

Message:

Fix QWebHistory serialization.

Patch by Jedrzej Nowacki <​jedrzej.nowacki@nokia.com> on 2010-05-18
Reviewed by Simon Hausmann.

Regression was caused by bug 33224. The streaming function
should generate a documentSequenceNumber for all loaded values.

[Qt] tst_QWebHistory::serialize_2() fails
​https://bugs.webkit.org/show_bug.cgi?id=37322

• Api/qwebhistory.cpp:

(operator>>):

Location:

trunk/WebKit/qt

Files:

• Api/qwebhistory.cpp (modified) (1 diff)
• ChangeLog (modified) (1 diff)

trunk/WebKit/qt/Api/qwebhistory.cpp

@@ -541 +541 @@
             }
             d->lst->removeItem(nullItem);
+            // Update the HistoryController.
+            history.d->lst->page()->mainFrame()->loader()->history()->setCurrentItem(history.d->lst->entries()[currentIndex].get());
             history.goToItem(history.itemAt(currentIndex));
         }

trunk/WebKit/qt/ChangeLog

@@ +1 @@
+2010-05-18  Jedrzej Nowacki  <jedrzej.nowacki@nokia.com>
+
+        Reviewed by Simon Hausmann.
+
+        Fix QWebHistory serialization.
+
+        Regression was caused by bug 33224. The streaming function
+        should generate a documentSequenceNumber for all loaded values.
+
+        [Qt] tst_QWebHistory::serialize_2() fails
+        https://bugs.webkit.org/show_bug.cgi?id=37322
+
+        * Api/qwebhistory.cpp:
+        (operator>>):
+
 2010-05-17  Tasuku Suzuki  <tasuku.suzuki@nokia.com>