Closed
Bug 809165
Opened 12 years ago
Closed 12 years ago
Add a updatedTime field to the application object
Categories
(Core Graveyard :: DOM: Apps, defect)
Core Graveyard
DOM: Apps
Tracking
(blocking-basecamp:+, firefox18 fixed, firefox19 fixed)
People
(Reporter: fabrice, Assigned: fabrice)
References
Details
(Whiteboard: [qa-])
Attachments
(1 file)
|
(deleted),
patch
|
philikon
:
review+
sicking
:
superreview+
|
Details | Diff | Splinter Review |
It will initially have the same value as installTime, and then be updated when... updating.
|
Assignee | |
Updated•12 years ago
|
blocking-basecamp: --- → ?
We need this for a high-performance home-screen which also always keeps icons up-to-date.
blocking-basecamp: ? → +
|
|
Assignee | |
Comment 2•12 years ago
|
||
With this patch we get the updateTime property updated on hosted apps when the new manifest is fetched, and on packaged apps once the new package is installed.
The default value of the property is 0.
Assignee: nobody → fabrice
|
||
Comment 3•12 years ago
|
||
Comment on attachment 678907 [details] [diff] [review]
patch
Review of attachment 678907 [details] [diff] [review]:
-----------------------------------------------------------------
::: dom/apps/src/Webapps.js
@@ +368,5 @@
> this.removable = aApp.removable;
> this.lastUpdateCheck = aApp.lastUpdateCheck ? aApp.lastUpdateCheck
> : Date.now();
> + this.updateTime = aApp.updateTime ? aApp.updateTime
> + : 0;
Make the fall back `aApp.installTime` please. r=me with that.
Attachment #678907 -
Flags: review+
|
|
||
Updated•12 years ago
|
Attachment #678907 -
Flags: superreview?(jonas)
Attachment #678907 -
Flags: superreview?(jonas) → superreview+
|
|
Assignee | |
Comment 4•12 years ago
|
||
|
||
Comment 5•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
|
|
||
Comment 6•12 years ago
|
||
status-firefox18:
--- → fixed
status-firefox19:
--- → fixed
|
||
Updated•7 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•