Closed Bug 912164 Opened 6 years ago Closed 6 years ago

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

Categories

(Firefox OS Graveyard :: General, defect)

x86_64
Linux
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: marco, Assigned: marco)

References

Details

Attachments

(1 file)

Attached patch setTimestampSplinter Review
The patch in bug 835799 introduced a timestamp property to the launch call. Now, this timestamp property isn't being set properly.
Attachment #799031 - Flags: review?(ferjmoreno)
Blocks: 844227
Comment on attachment 799031 [details] [diff] [review]
setTimestamp

Is this property still needed? If it is, I'll ask Fabrice to review it.
Attachment #799031 - Flags: review?(ferjmoreno)
Flags: needinfo?(21)
(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 :)
Flags: needinfo?(21)
Attachment #799031 - Flags: review?(fabrice)
Attachment #799031 - Flags: review?(fabrice) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/3b9b584fbf08
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Assignee: nobody → mar.castelluccio
You need to log in before you can comment on or make changes to this bug.