Changeset 117069 in webkit

Timestamp:

May 15, 2012 8:07:59 AM (12 years ago)

Author:

commit-queue@webkit.org

Message:

[Qt][WK2] Fix scrolling in touch mode
​https://bugs.webkit.org/show_bug.cgi?id=75006

Patch by Allan Sandfeld Jensen <​allan.jensen@nokia.com> on 2012-05-15
Reviewed by Kenneth Rohde Christiansen.

Source/WebCore:

• page/FrameView.cpp:

(WebCore::FrameView::setFixedVisibleContentRect):

On changes to fixedVisibleContentRect update scroll-animators position, and
recalculate page-step sizes in scrollbars.

• platform/ScrollView.cpp:

(WebCore::ScrollView::scrollTo):

Forward scrolling-request to the host window.

(WebCore::ScrollView::updateScrollbars):

Allow ScrollViews to have invisible scrollbars when scrolling is delegated.

• platform/ScrollView.h:

(ScrollView):

Source/WebKit2:

Only handle mouse-wheel events in WebCore to avoid double scrolling, and
scale pixels scrolled by wheel events according to viewport transformation.

• Shared/qt/WebEventFactoryQt.cpp:

(WebKit::WebEventFactory::createWebWheelEvent):

• UIProcess/qt/QtViewportInteractionEngine.cpp:
• UIProcess/qt/QtViewportInteractionEngine.h:

(QtViewportInteractionEngine):

• UIProcess/qt/QtWebPageEventHandler.cpp:

(QtWebPageEventHandler::handleWheelEvent):

Location:

trunk/Source

Files:

• WebCore/ChangeLog (modified) (1 diff)
• WebCore/page/FrameView.cpp (modified) (3 diffs)
• WebCore/platform/ScrollView.cpp (modified) (2 diffs)
• WebCore/platform/ScrollView.h (modified) (2 diffs)
• WebKit2/ChangeLog (modified) (1 diff)
• WebKit2/Shared/qt/WebEventFactoryQt.cpp (modified) (2 diffs)
• WebKit2/UIProcess/qt/QtViewportInteractionEngine.cpp (modified) (1 diff)
• WebKit2/UIProcess/qt/QtViewportInteractionEngine.h (modified) (1 diff)
• WebKit2/UIProcess/qt/QtWebPageEventHandler.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-05-15  Allan Sandfeld Jensen  <allan.jensen@nokia.com>
+
+        [Qt][WK2] Fix scrolling in touch mode
+        https://bugs.webkit.org/show_bug.cgi?id=75006
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        * page/FrameView.cpp:
+        (WebCore::FrameView::setFixedVisibleContentRect):
+            On changes to fixedVisibleContentRect update scroll-animators position, and
+            recalculate page-step sizes in scrollbars.
+        * platform/ScrollView.cpp:
+        (WebCore::ScrollView::scrollTo):
+            Forward scrolling-request to the host window.
+        (WebCore::ScrollView::updateScrollbars):
+            Allow ScrollViews to have invisible scrollbars when scrolling is delegated.
+        * platform/ScrollView.h:
+        (ScrollView):
+
 2012-05-15  Vsevolod Vlasov  <vsevik@chromium.org>
 

trunk/Source/WebCore/page/FrameView.cpp

@@ -1723 +1723 @@
 void FrameView::setFixedVisibleContentRect(const IntRect& visibleContentRect)
 {
+    bool visibleContentSizeDidChange = false;
     if (visibleContentRect.size() != this->fixedVisibleContentRect().size()) {
         // When the viewport size changes or the content is scaled, we need to
@@ -1728 +1729 @@
         if (RenderView* root = rootRenderer(this))
             root->setFixedPositionedObjectsNeedLayout();
+        visibleContentSizeDidChange = true;
     }
 
@@ -1735 +1737 @@
         if (m_frame->page()->settings()->acceleratedCompositingForFixedPositionEnabled())
             updateFixedElementsAfterScrolling();
+        scrollAnimator()->setCurrentPosition(scrollPosition());
         scrollPositionChanged();
+    }
+    if (visibleContentSizeDidChange) {
+        // Update the scroll-bars to calculate new page-step size.
+        updateScrollbars(scrollOffset());
     }
     frame()->loader()->client()->didChangeScrollOffset();

trunk/Source/WebCore/platform/ScrollView.cpp

@@ -372 +372 @@
 
     repaintFixedElementsAfterScrolling();
+#if USE(TILED_BACKING_STORE)
+    if (delegatesScrolling()) {
+        hostWindow()->delegatedScrollRequested(IntPoint(newOffset));
+        return;
+    }
+#endif
     scrollContents(scrollDelta);
     updateFixedElementsAfterScrolling();
@@ -454 +460 @@
 void ScrollView::updateScrollbars(const IntSize& desiredOffset)
 {
-    if (m_inUpdateScrollbars || prohibitsScrolling() || delegatesScrolling() || platformWidget())
+    if (m_inUpdateScrollbars || prohibitsScrolling() || platformWidget())
         return;
 

trunk/Source/WebCore/platform/ScrollView.h

@@ -314 +314 @@
     virtual bool isFlippedDocument() const { return false; }
 
+    // Called to update the scrollbars to accurately reflect the state of the view.
+    void updateScrollbars(const IntSize& desiredOffset);
+
 private:
     RefPtr<Scrollbar> m_horizontalScrollbar;
@@ -356 +359 @@
     void destroy();
 
-    // Called to update the scrollbars to accurately reflect the state of the view.
-    void updateScrollbars(const IntSize& desiredOffset);
     IntRect rectToCopyOnScroll() const;
 

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2012-05-15  Allan Sandfeld Jensen  <allan.jensen@nokia.com>
+
+        [Qt][WK2] Fix scrolling in touch mode
+        https://bugs.webkit.org/show_bug.cgi?id=75006
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        Only handle mouse-wheel events in WebCore to avoid double scrolling, and
+        scale pixels scrolled by wheel events according to viewport transformation.
+
+        * Shared/qt/WebEventFactoryQt.cpp:
+        (WebKit::WebEventFactory::createWebWheelEvent):
+        * UIProcess/qt/QtViewportInteractionEngine.cpp:
+        * UIProcess/qt/QtViewportInteractionEngine.h:
+        (QtViewportInteractionEngine):
+        * UIProcess/qt/QtWebPageEventHandler.cpp:
+        (QtWebPageEventHandler::handleWheelEvent):
+
 2012-05-15  Kenneth Rohde Christiansen  <kenneth@webkit.org>
 

trunk/Source/WebKit2/Shared/qt/WebEventFactoryQt.cpp

@@ -28 +28 @@
 #include "WebEventFactoryQt.h"
 #include <QKeyEvent>
+#include <QLineF>
 #include <QTransform>
+#include <WebCore/FloatPoint.h>
+#include <WebCore/FloatSize.h>
 #include <WebCore/IntPoint.h>
-#include <WebCore/FloatPoint.h>
 #include <WebCore/PlatformKeyboardEvent.h>
 #include <wtf/ASCIICType.h>
@@ -150 +152 @@
     deltaY = wheelTicksY * wheelScrollLines * cDefaultQtScrollStep;
 
-    return WebWheelEvent(WebEvent::Wheel, fromItemTransform.map(e->posF()).toPoint(), e->globalPosF().toPoint(), FloatSize(deltaX, deltaY), FloatSize(wheelTicksX, wheelTicksY), granularity, modifiers, timestamp);
+    // Transform the position and the pixel scrolling distance.
+    QLineF transformedScroll = fromItemTransform.map(QLineF(e->posF(), e->posF() + QPointF(deltaX, deltaY)));
+    IntPoint transformedPoint = transformedScroll.p1().toPoint();
+    IntPoint globalPoint = e->globalPosF().toPoint();
+    FloatSize transformedDelta(transformedScroll.dx(), transformedScroll.dy());
+    FloatSize wheelTicks(wheelTicksX, wheelTicksY);
+    return WebWheelEvent(WebEvent::Wheel, transformedPoint, globalPoint, transformedDelta, wheelTicks, granularity, modifiers, timestamp);
 }
 

trunk/Source/WebKit2/UIProcess/qt/QtViewportInteractionEngine.cpp

@@ -262 +262 @@
 }
 
-void QtViewportInteractionEngine::wheelEvent(QWheelEvent* ev)
-{
-    if (scrollAnimationActive() || scaleAnimationActive() || pinchGestureActive())
-        return; // Ignore.
-
-
-    // A normal scroll-tick should have a delta of 120 (1/8) degrees. Convert this to
-    // local standard scroll step of 3 lines of 20 pixels.
-    static const int cDefaultQtScrollStep = 20;
-    static const int wheelScrollLines = 3;
-    const int wheelTick = wheelScrollLines * cDefaultQtScrollStep;
-
-    int pixelDelta = ev->delta() * (wheelTick / 120.f);
-
-    QPointF newPosition = m_viewport->contentPos();
-
-    if (ev->orientation() == Qt::Horizontal)
-        newPosition.rx() -= pixelDelta;
-    else
-        newPosition.ry() -= pixelDelta;
-
-    QRectF endPosRange = computePosRangeForItemAtScale(m_content->contentsScale());
-
-    QPointF currentPosition = m_viewport->contentPos();
-    newPosition = boundPosition(endPosRange.topLeft(), newPosition, endPosRange.bottomRight());
-    m_viewport->setContentPos(newPosition);
-
-    emit contentViewportChanged(currentPosition - newPosition);
-}
-
 void QtViewportInteractionEngine::pagePositionRequest(const QPoint& pagePosition)
 {

trunk/Source/WebKit2/UIProcess/qt/QtViewportInteractionEngine.h

@@ -66 +66 @@
     bool animateItemRectVisible(const QRectF&);
 
-    void wheelEvent(QWheelEvent*);
     void pagePositionRequest(const QPoint& pos);
     void touchBegin();

trunk/Source/WebKit2/UIProcess/qt/QtWebPageEventHandler.cpp

@@ -155 +155 @@
     QTransform fromItemTransform = m_webPage->transformFromItem();
     m_webPageProxy->handleWheelEvent(NativeWebWheelEvent(ev, fromItemTransform));
-    // FIXME: Handle whether the page used the wheel event or not.
-    if (m_interactionEngine)
-        m_interactionEngine->wheelEvent(ev);
 }