400493 - Track download finish time in Download UI

Status: VERIFIED FIXED

(Toolkit :: Downloads API, enhancement)

Description

[Ed Lee :Mardak]

Splitting patch off from bug 223895 which isn't needed for that patch anymore, but we'll probably need this later on if we want to show something with the completed time (instead of just the start time).

Comment 1

[Ed Lee :Mardak]

Attachment: v1

Not a critical patch to review.. Simply sets the download finish time to Now when downloads finish or use the time from the DB.

Comment 2

[Shawn Wilsher :sdwilsh]

Comment on attachment 285567 [details] [diff] [review]
v1

r=sdwilsh

This is a low risk patch

Comment 3

[Mike Connor [:mconnor]]

Comment on attachment 285567 [details] [diff] [review]
v1

not needed until other UI pieces are ready, defer until then

Comment 4

[Ed Lee :Mardak]

Checking in toolkit/mozapps/downloads/content/downloads.js;
/cvsroot/mozilla/toolkit/mozapps/downloads/content/downloads.js,v  <--  downloads.js
new revision: 1.101; previous revision: 1.100
done

Comment 5

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED (in the backend), since I've verified the frontend components already.