Add time stamp to the constructor of the events

third_party/WebKit/Source/core/events/PointerEventFactoryTest.cpp

Index: third_party/WebKit/Source/core/events/PointerEventFactoryTest.cpp
diff --git a/third_party/WebKit/Source/core/events/PointerEventFactoryTest.cpp b/third_party/WebKit/Source/core/events/PointerEventFactoryTest.cpp
index 08c26490f949ab9bce0451a46848598103ce8d53..4e39734016d5c817bcfc07acd3a2e16fc6b659cd 100644
--- a/third_party/WebKit/Source/core/events/PointerEventFactoryTest.cpp
+++ b/third_party/WebKit/Source/core/events/PointerEventFactoryTest.cpp
@@ -70,7 +70,8 @@ class PointerEventFactoryTest : public ::testing::Test {
    public:
     WebMouseEventBuilder(WebPointerProperties::PointerType,
                          int,
-                         WebInputEvent::Modifiers);
+                         WebInputEvent::Modifiers,
+                         double);
   };
 };
 
@@ -92,11 +93,13 @@ PointerEventFactoryTest::WebTouchPointBuilder::WebTouchPointBuilder(
 PointerEventFactoryTest::WebMouseEventBuilder::WebMouseEventBuilder(
     WebPointerProperties::PointerType pointer_type_param,
     int id_param,
-    WebInputEvent::Modifiers modifiers_param) {
+    WebInputEvent::Modifiers modifiers_param,
+    double platform_time_stamp) {
   pointer_type = pointer_type_param;
   id = id_param;
   modifiers_ = modifiers_param;
   frame_scale_ = 1;
+  time_stamp_seconds_ = platform_time_stamp;
 }
 
 PointerEvent* PointerEventFactoryTest::CreateAndCheckTouchCancel(
@@ -104,12 +107,14 @@ PointerEvent* PointerEventFactoryTest::CreateAndCheckTouchCancel(
     int raw_id,
     int unique_id,
     bool is_primary) {
-  PointerEvent* pointer_event =
-      pointer_event_factory_.CreatePointerCancelEvent(unique_id, pointer_type);
+  TimeTicks now = TimeTicks::Now();
+  PointerEvent* pointer_event = pointer_event_factory_.CreatePointerCancelEvent(
+      unique_id, pointer_type, now);
   EXPECT_EQ(unique_id, pointer_event->pointerId());
   EXPECT_EQ(is_primary, pointer_event->isPrimary());
   EXPECT_EQ(PointerTypeNameForWebPointPointerType(pointer_type),
             pointer_event->pointerType());
+  EXPECT_EQ(now, pointer_event->PlatformTimeStamp());
   return pointer_event;
 }
 
@@ -132,17 +137,21 @@ PointerEvent* PointerEventFactoryTest::CreateAndCheckTouchEvent(
     bool is_primary,
     WebTouchPoint::State state,
     size_t coalesced_event_count) {
-  Vector<WebTouchPoint> coalesced_events;
+  Vector<std::pair<WebTouchPoint, TimeTicks>> coalesced_events;
+  TimeTicks now = TimeTicks::Now();
   for (size_t i = 0; i < coalesced_event_count; i++) {
-    coalesced_events.push_back(PointerEventFactoryTest::WebTouchPointBuilder(
-        pointer_type, raw_id, state));
+    coalesced_events.push_back(std::pair<WebTouchPoint, TimeTicks>(
+        PointerEventFactoryTest::WebTouchPointBuilder(pointer_type, raw_id,
+                                                      state),
+        now));
   }
   PointerEvent* pointer_event = pointer_event_factory_.Create(
       PointerEventFactoryTest::WebTouchPointBuilder(pointer_type, raw_id,
                                                     state),
-      coalesced_events, WebInputEvent::kNoModifiers, nullptr, nullptr);
+      coalesced_events, WebInputEvent::kNoModifiers, now, nullptr, nullptr);
   EXPECT_EQ(unique_id, pointer_event->pointerId());
   EXPECT_EQ(is_primary, pointer_event->isPrimary());
+  EXPECT_EQ(now, pointer_event->PlatformTimeStamp());
   const char* expected_pointer_type =
       PointerTypeNameForWebPointPointerType(pointer_type);
   EXPECT_EQ(expected_pointer_type, pointer_event->pointerType());
@@ -151,6 +160,7 @@ PointerEvent* PointerEventFactoryTest::CreateAndCheckTouchEvent(
     EXPECT_EQ(unique_id, pointer_event->getCoalescedEvents()[i]->pointerId());
     EXPECT_EQ(is_primary, pointer_event->getCoalescedEvents()[i]->isPrimary());
     EXPECT_EQ(expected_pointer_type, pointer_event->pointerType());
+    EXPECT_EQ(now, pointer_event->PlatformTimeStamp());
   }
   return pointer_event;
 }
@@ -163,18 +173,21 @@ PointerEvent* PointerEventFactoryTest::CreateAndCheckMouseEvent(
     WebInputEvent::Modifiers modifiers,
     size_t coalesced_event_count) {
   Vector<WebMouseEvent> coalesced_events;
+  double base_time = 123;

[dtapuska, 2017/04/24 15:27:43]

Can we use
https://cs.chromium.org/chromium/src/third_party/WebKit/public/platform/WebIn...
here?

[Navid Zolghadr, 2017/04/24 15:51:45]

Makes sense. Done.

   for (size_t i = 0; i < coalesced_event_count; i++) {
     coalesced_events.push_back(PointerEventFactoryTest::WebMouseEventBuilder(
-        pointer_type, raw_id, modifiers));
+        pointer_type, raw_id, modifiers, 123 + i));

[dtapuska, 2017/04/24 15:27:43]

Can this be base_time + i (instead of 123 + i) ?

[Navid Zolghadr, 2017/04/24 15:51:45]

Done.

   }
   PointerEvent* pointer_event = pointer_event_factory_.Create(
       coalesced_event_count ? EventTypeNames::mousemove
                             : EventTypeNames::mousedown,
       PointerEventFactoryTest::WebMouseEventBuilder(pointer_type, raw_id,
-                                                    modifiers),
+                                                    modifiers, base_time),
       coalesced_events, nullptr);
   EXPECT_EQ(unique_id, pointer_event->pointerId());
   EXPECT_EQ(is_primary, pointer_event->isPrimary());
+  EXPECT_EQ(TimeTicks::FromSeconds(base_time),
+            pointer_event->PlatformTimeStamp());
   const char* expected_pointer_type =
       PointerTypeNameForWebPointPointerType(pointer_type);
   EXPECT_EQ(expected_pointer_type, pointer_event->pointerType());
@@ -183,6 +196,8 @@ PointerEvent* PointerEventFactoryTest::CreateAndCheckMouseEvent(
     EXPECT_EQ(unique_id, pointer_event->getCoalescedEvents()[i]->pointerId());
     EXPECT_EQ(is_primary, pointer_event->getCoalescedEvents()[i]->isPrimary());
     EXPECT_EQ(expected_pointer_type, pointer_event->pointerType());
+    EXPECT_EQ(TimeTicks::FromSeconds(base_time + i),
+              pointer_event->getCoalescedEvents()[i]->PlatformTimeStamp());
   }
   return pointer_event;
 }