912164 - Timestamp property is no longer set for app.launch()

Status: RESOLVED FIXED

(Firefox OS Graveyard :: General, defect)

Description

[Marco Castelluccio [:marco]]

Attachment: setTimestamp

The patch in bug 835799 introduced a timestamp property to the launch call. Now, this timestamp property isn't being set properly.

Comment 1

[Marco Castelluccio [:marco]]

Comment on attachment 799031 [details] [diff] [review]
setTimestamp

Is this property still needed? If it is, I'll ask Fabrice to review it.

Comment 2

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to Marco Castelluccio [:marco] from comment #1)
> Comment on attachment 799031 [details] [diff] [review]
> setTimestamp
> 
> Is this property still needed? If it is, I'll ask Fabrice to review it.

It is useful to get the delay between the actual click on the icon and the time needed to receive this information in the system app. So please keep it :)

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/b2g-inbound/rev/3b9b584fbf08

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/3b9b584fbf08