915658 - Add timestamp data to touch event logging

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, defect)

Description

[Malini Das [:mdas] - Away, not checking bugmail]

When we dispatch touch events, we detail the coordinates, but not the timestamp data. The timestamp is needed for getting performance numbers.

Comment 1

[Malini Das [:mdas] - Away, not checking bugmail]

Attachment: datetouch

Prepends log lines with date

Comment 2

[Malini Das [:mdas] - Away, not checking bugmail]

Attachment: add date

That last patch was silly.

Comment 3

[Jonathan Griffin (:jgriffin)]

Comment on attachment 803807 [details] [diff] [review]
add date

Review of attachment 803807 [details] [diff] [review]:
-----------------------------------------------------------------

::: testing/marionette/marionette-listener.js
@@ +283,5 @@
>    mouseEventsOnly = false;
>  }
>  
> +/**
> + * Dump a logline to logcat. Prepends logline with a timestamp.

s/logcat/stdout (which happens to go to logcat on B2G device builds)

Comment 4

[Malini Das [:mdas] - Away, not checking bugmail]

landed on inbound:
https://hg.mozilla.org/integration/mozilla-inbound/rev/62f6766d63b8

Comment 5

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/62f6766d63b8

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-b2g18/rev/b459ac3ebe24

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-b2g18_v1_1_0_hd/rev/b459ac3ebe24