Changeset 86090 in webkit

Timestamp:

May 9, 2011 2:56:06 PM (13 years ago)

Author:

commit-queue@webkit.org

Message:

2011-05-09 Igor Oliveira <​igor.oliveira@openbossa.org>

Reviewed by Andreas Kling.

[Qt] DRT eventSender.addTouchPoint method is not setting the start position
​https://bugs.webkit.org/show_bug.cgi?id=60487

When adding a new touch point, the start position of this point needs to be set.

• DumpRenderTree/qt/EventSenderQt.cpp: (EventSender::addTouchPoint):

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• DumpRenderTree/qt/EventSenderQt.cpp (modified) (1 diff)

trunk/Tools/ChangeLog

@@ +1 @@
+2011-05-09  Igor Oliveira  <igor.oliveira@openbossa.org>
+
+        Reviewed by Andreas Kling.
+
+        [Qt] DRT eventSender.addTouchPoint method is not setting the start position
+        https://bugs.webkit.org/show_bug.cgi?id=60487
+
+        When adding a new touch point, the start position of this point needs to be set.
+
+        * DumpRenderTree/qt/EventSenderQt.cpp:
+        (EventSender::addTouchPoint):
+
 2011-05-09  Robert Hogan  <robert@webkit.org>
 

trunk/Tools/DumpRenderTree/qt/EventSenderQt.cpp

@@ -395 +395 @@
 void EventSender::addTouchPoint(int x, int y)
 {
-    // Use index to refer to the position in the vector that this touch
-    // is stored. We then create a unique id for the touch that will be
-    // passed into WebCore.
-    int index = m_touchPoints.count();
-    int id = m_touchPoints.isEmpty() ? 0 : m_touchPoints.last().id() + 1;
+    const int id = m_touchPoints.isEmpty() ? 0 : m_touchPoints.last().id() + 1;
+    const QPointF pos(x, y);
     QTouchEvent::TouchPoint point(id);
+    point.setPos(pos);
+    point.setStartPos(pos);
+    point.setState(Qt::TouchPointPressed);
     m_touchPoints.append(point);
-    updateTouchPoint(index, x, y);
-    m_touchPoints[index].setState(Qt::TouchPointPressed);
 }