Add a timestamp property to the observer sent when an application has been called by app.launch() in order to have a better measure

RESOLVED FIXED in Firefox 21

Status

defect
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: vingtetun, Assigned: vingtetun)

Tracking

({perf})

unspecified
B2G C4 (2jan on)
x86_64
Linux
Dependency tree / graph
Bug Flags:
in-moztrap -

Firefox Tracking Flags

(blocking-b2g:tef+, firefox19 wontfix, firefox20 wontfix, firefox21 fixed, b2g18+ fixed, b2g18-v1.0.0 fixed, b2g18-v1.0.1 fixed)

Details

(Whiteboard: [FFOS_perf] QARegressExclude)

Attachments

(1 attachment)

Posted patch PatchSplinter Review
The patch does what the title says.
Attachment #707611 - Flags: review?(fabrice)
Assignee: nobody → 21
blocking-b2g: --- → tef?
Will consider uplift after an approval nomination, but this is a non-blocker.
blocking-b2g: tef? → -
tracking-b2g18: --- → +
Attachment #707611 - Flags: review?(fabrice) → review+
Comment on attachment 707611 [details] [diff] [review]
Patch

This patch is really non-risky: it does not change any logic nor does and just add a new property on an element.

If we don't take this patch the value reported on the performance infrastructure will not take into account some time spend in Gecko before launching an applicaton... So this patch will regress performance on the dashboard but that's to be more precise...
Attachment #707611 - Flags: approval-mozilla-b2g18?
Comment on attachment 707611 [details] [diff] [review]
Patch

Approving for 1.0.0 (mozilla-b2g18_v_1_0_0 branch tip) uplift.
Attachment #707611 - Flags: approval-mozilla-b2g18? → approval-mozilla-b2g18+
Keywords: perf
If you want assistance getting this landed, feel free to set the checkin-needed flag up top.
https://hg.mozilla.org/mozilla-central/rev/47c28fd47df9
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Marking this tef+ so it can be uplifted to v1.0.1 branch as it appears to have missed the 1/25 merge from v1-train and so is not on v1.0.1 as was expected.
blocking-b2g: - → tef+
This landed on b2g18 before v1_0_1 was forked. Note the cset is identical to the b2g18 one in comment 7.
https://hg.mozilla.org/releases/mozilla-b2g18_v1_0_1/rev/445db979f18d
Whiteboard: [FFOS_perf] → [FFOS_perf] QARegressExclude
No Test case creation is needed in moztrap for this issue.
Flags: in-moztrap-
Cannot verify, need steps to blackbox test this issue.
Depends on: 912164
You need to log in before you can comment on or make changes to this bug.