Closed
Bug 665260
Opened 13 years ago
Closed 13 years ago
Store start time in sessionstore data
Categories
(Firefox :: Session Restore, defect)
Firefox
Session Restore
Tracking
()
RESOLVED
FIXED
Firefox 7
People
(Reporter: zpao, Assigned: zpao)
References
Details
Attachments
(1 file)
3.73 KB,
patch
|
dietrich
:
review+
|
Details | Diff | Splinter Review |
I think it could be interesting to have this data for telemetry, even if it doesn't have any practical uses. To be clear, I mean total age, not uptime of current process. The way we write sessionstore.js, it clobbers the file creation time stamp each write so we can't just extract it. Dietrich, Taras - does this sound useful?
Comment 1•13 years ago
|
||
(In reply to comment #0) > Dietrich, Taras - does this sound useful? Yes.
Assignee | ||
Comment 2•13 years ago
|
||
Does the thing. Due to bug 665702, this data isn't exposed outside of the file. I tested by checking the timestamp in sessionstore.js like so: first startup with apptabs (no previous value): 1308595047885 restart (session not resumed / just app tabs): 1308595192604 restore previous session: 1308595047885 (the start time from previous session overwrites new value) restart with resume_session_once=true: 1308595047885 restart with startup.page=3 (aka restore): 1308595047885
Comment 3•13 years ago
|
||
Comment on attachment 540595 [details] [diff] [review] Patch v0.1 Review of attachment 540595 [details] [diff] [review]: ----------------------------------------------------------------- looks a-ok, r=me.
Attachment #540595 -
Flags: review?(dietrich) → review+
Assignee | ||
Updated•13 years ago
|
Summary: Store session age in sessionstore data → Store start time in sessionstore data
Whiteboard: [inbound]
Comment 4•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/05b7d31f72bb
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → Firefox 7
You need to log in
before you can comment on or make changes to this bug.
Description
•