Add preventPropagation bit to GestureScrollUpdate event

Source/web/tests/WebInputEventConversionTest.cpp

Index: Source/web/tests/WebInputEventConversionTest.cpp
diff --git a/Source/web/tests/WebInputEventConversionTest.cpp b/Source/web/tests/WebInputEventConversionTest.cpp
index 810a1e4b84c3560f4727fa81d4f3490ae1bb7133..14989c7d0bb25da406de2b479fddfb460d45ce64 100644
--- a/Source/web/tests/WebInputEventConversionTest.cpp
+++ b/Source/web/tests/WebInputEventConversionTest.cpp
@@ -391,7 +391,7 @@ TEST(WebInputEventConversionTest, InputEventsScaling)
     }
 
     {
-        PlatformGestureEvent platformGestureEvent(PlatformEvent::GestureScrollUpdate, IntPoint(10, 10), IntPoint(10, 10), IntSize(10, 10), 0, false, false, false, false, 10, 10, 10, 10);
+        PlatformGestureEvent platformGestureEvent(PlatformEvent::GestureScrollUpdate, IntPoint(10, 10), IntPoint(10, 10), IntSize(10, 10), 0, false, false, false, false, 10, 10, 10, 10, false);

[Rick Byers, 2014/11/14 18:21:45]

You can add a conversion unit test now that sets this bit to true, right?  Might
as well do that.

[majidvp, 2014/11/17 21:33:03]

Good point. The conversion from PlatformGestureEvent to GestureEvent actually
drops the preventPropagation bit (same for velocityX, velocityY fields). So the
flag is never transferred to the final WebGestureEvent object. I modified the
test to set the initial flag as 'true' and assert that the final value is indeed
'false'. I think testing the false->false case does not add any additional value
as it verifies a trivial initialization logic which is already covered by this
test too.

Also updated the test covering the reverse conversion (WebGestureEvent ->
PlatformGestureEvent) to set and verify the preventPropagation flag to a non
default value 'true'.

         RefPtrWillBeRawPtr<GestureEvent> gestureEvent = GestureEvent::create(domWindow, platformGestureEvent);
         WebGestureEventBuilder webGestureBuilder(view, documentRenderView, *gestureEvent);