Title: [120202] trunk/Source/WebKit2
Revision
120202
Author
[email protected]
Date
2012-06-13 07:22:13 -0700 (Wed, 13 Jun 2012)

Log Message

[Qt] Do not set contents pos to the current position
https://bugs.webkit.org/show_bug.cgi?id=88990

Reviewed by Laszlo Gombos.

This is basically an optimization but it also solves flickering
in some cases.

* UIProcess/API/qt/qquickwebview.cpp:
(QQuickWebView::setContentPos):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (120201 => 120202)


--- trunk/Source/WebKit2/ChangeLog	2012-06-13 14:08:46 UTC (rev 120201)
+++ trunk/Source/WebKit2/ChangeLog	2012-06-13 14:22:13 UTC (rev 120202)
@@ -1,3 +1,16 @@
+2012-06-13  Kenneth Rohde Christiansen  <[email protected]>
+
+        [Qt] Do not set contents pos to the current position
+        https://bugs.webkit.org/show_bug.cgi?id=88990
+
+        Reviewed by Laszlo Gombos.
+
+        This is basically an optimization but it also solves flickering
+        in some cases.
+
+        * UIProcess/API/qt/qquickwebview.cpp:
+        (QQuickWebView::setContentPos):
+
 2012-06-12  MORITA Hajime  <[email protected]>
 
         Shadow Pseudo ID should be able to nest to point nested shadow DOM.

Modified: trunk/Source/WebKit2/UIProcess/API/qt/qquickwebview.cpp (120201 => 120202)


--- trunk/Source/WebKit2/UIProcess/API/qt/qquickwebview.cpp	2012-06-13 14:08:46 UTC (rev 120201)
+++ trunk/Source/WebKit2/UIProcess/API/qt/qquickwebview.cpp	2012-06-13 14:22:13 UTC (rev 120202)
@@ -1799,6 +1799,10 @@
 void QQuickWebView::setContentPos(const QPointF& pos)
 {
     Q_D(QQuickWebView);
+
+    if (pos == contentPos())
+        return;
+
     d->setContentPos(pos);
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to