Closed Bug 915658 Opened 8 years ago Closed 8 years ago

Add timestamp data to touch event logging

Categories

(Testing :: Marionette, defect)

defect
Not set
normal

Tracking

(firefox24 wontfix, firefox25 wontfix, firefox26 fixed, b2g18 fixed, b2g18-v1.0.0 wontfix, b2g18-v1.0.1 wontfix, b2g-v1.1hd fixed)

RESOLVED FIXED
mozilla26
Tracking Status
firefox24 --- wontfix
firefox25 --- wontfix
firefox26 --- fixed
b2g18 --- fixed
b2g18-v1.0.0 --- wontfix
b2g18-v1.0.1 --- wontfix
b2g-v1.1hd --- fixed

People

(Reporter: mdas, Assigned: mdas)

References

Details

(Whiteboard: [c=power p= s= u=])

Attachments

(1 file, 1 obsolete file)

When we dispatch touch events, we detail the coordinates, but not the timestamp data. The timestamp is needed for getting performance numbers.
Blocks: 915663
Blocks: 915679
Attached patch datetouch (obsolete) — Splinter Review
Prepends log lines with date
Attachment #803797 - Flags: review?(jgriffin)
Attached patch add dateSplinter Review
That last patch was silly.
Attachment #803797 - Attachment is obsolete: true
Attachment #803797 - Flags: review?(jgriffin)
Attachment #803807 - Flags: review?(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)
Attachment #803807 - Flags: review?(jgriffin) → review+
Whiteboard: [c=power p= s= u=]
https://hg.mozilla.org/mozilla-central/rev/62f6766d63b8
Assignee: nobody → mdas
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
Whiteboard: [c=power p= s= u=] → [c=power p= s= u=][checkin-needed-b2g18]
You need to log in before you can comment on or make changes to this bug.