Update the EventSendingController to send complete touch events.

webkit/tools/test_shell/event_sending_controller.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This file contains the definition for EventSendingController.
 //
 // Some notes about drag and drop handling:
 // Windows drag and drop goes through a system call to DoDragDrop.  At that
 // point, program control is given to Windows which then periodically makes
 // callbacks into the webview.  This won't work for layout tests, so instead,
@@ skipping 822 matching lines @@
   result->SetNull();
 }
 
 void EventSendingController::addTouchPoint(
     const CppArgumentList& args, CppVariant* result) {
   result->SetNull();
 
   WebTouchPoint touch_point;
   touch_point.state = WebTouchPoint::StatePressed;
   touch_point.position = WebPoint(args[0].ToInt32(), args[1].ToInt32());
+  touch_point.screenPosition = touch_point.position;
   touch_point.id = touch_points.size();
   touch_points.push_back(touch_point);
 }
 
 void EventSendingController::clearTouchPoints(
     const CppArgumentList& args, CppVariant* result) {
   result->SetNull();
 
   touch_points.clear();
 }
@@ skipping 41 matching lines @@
   const unsigned int index = args[0].ToInt32();
   if (index >= touch_points.size()) {
     NOTREACHED() << "Invalid touch point index";
   }
 
   WebPoint position(args[1].ToInt32(), args[2].ToInt32());
 
   WebTouchPoint* touch_point = &touch_points[index];
   touch_point->state = WebTouchPoint::StateMoved;
   touch_point->position = position;
+  touch_point->screenPosition = position;
 }
 
 void EventSendingController::cancelTouchPoint(
     const CppArgumentList& args, CppVariant* result) {
   result->SetNull();
 
   const unsigned int index = args[0].ToInt32();
   if (index >= touch_points.size()) {
     NOTREACHED() << "Invalid touch point index";
   }
 
   WebTouchPoint* touch_point = &touch_points[index];
   touch_point->state = WebTouchPoint::StateCancelled;
 }
 
 void EventSendingController::SendCurrentTouchEvent(
     const WebInputEvent::Type type) {
+    
+  // Force a layout here just to make sure every position has been
+  // determined before we send events (as well as all the other methods
+  // that send an event do).

[jamesr, 2010/11/10 19:59:58]

Comment isn't very helpful here, recommend removing it (see WebKit review
feedback).

+  webview()->layout();
+
   if (static_cast<unsigned int>(WebTouchEvent::touchPointsLengthCap) <=
       touch_points.size()) {
     NOTREACHED() << "Too many touch points for event";
   }
 
   WebTouchEvent touch_event;
   touch_event.type = type;
   touch_event.modifiers = touch_modifiers;
   touch_event.timeStampSeconds = GetCurrentEventTimeSec();
   touch_event.touchPointsLength = touch_points.size();
@@ skipping 82 matching lines @@
 
 void EventSendingController::fireKeyboardEventsToElement(
     const CppArgumentList& args, CppVariant* result) {
   result->SetNull();
 }
 
 void EventSendingController::clearKillRing(
     const CppArgumentList& args, CppVariant* result) {
   result->SetNull();
 }