The default bug view has changed. See this FAQ.

Store start time in sessionstore data

RESOLVED FIXED in Firefox 7

Status

()

Firefox
Session Restore
RESOLVED FIXED
6 years ago
4 years ago

People

(Reporter: zpao, Assigned: zpao)

Tracking

unspecified
Firefox 7
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

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

6 years ago
(In reply to comment #0)
> Dietrich, Taras - does this sound useful?

Yes.
Created attachment 540595 [details] [diff] [review]
Patch v0.1

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
Assignee: nobody → paul
Status: NEW → ASSIGNED
Attachment #540595 - Flags: review?(dietrich)
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+
Summary: Store session age in sessionstore data → Store start time in sessionstore data
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/05b7d31f72bb
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → Firefox 7

Comment 5

6 years ago
How can this be verified? 
Thx

Updated

6 years ago
Blocks: 698274
Blocks: 902729
You need to log in before you can comment on or make changes to this bug.