Closed
Bug 900776
Opened 12 years ago
Closed 12 years ago
Trace actor reports NaN time when exiting frames
Categories
(DevTools :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 25
People
(Reporter: rjacob, Assigned: rjacob)
References
Details
Attachments
(1 file, 1 obsolete file)
|
3.23 KB,
patch
|
fitzgen
:
review+
|
Details | Diff | Splinter Review |
When exiting a frame, the trace actor reports that the time since the trace started is NaN. It should report the time in the same format as it does on frame entry (currently, milliseconds since tracing started).
| Assignee | ||
Comment 1•12 years ago
|
||
Attachment #784698 -
Flags: review?(nfitzgerald)
| Assignee | ||
Updated•12 years ago
|
Attachment #784698 -
Flags: review?(nfitzgerald)
| Assignee | ||
Comment 2•12 years ago
|
||
Whoops, generated the patch wrong.
Attachment #784698 -
Attachment is obsolete: true
Attachment #784702 -
Flags: review?(nfitzgerald)
Comment 3•12 years ago
|
||
Comment on attachment 784702 [details] [diff] [review]
exitTime.diff
Review of attachment 784702 [details] [diff] [review]:
-----------------------------------------------------------------
r+ with green try push[0]
In the future, format your patches with 8 lines of context[1].
[0] https://wiki.mozilla.org/ReleaseEngineering/TryServer#How_to_push_to_try
[1] https://developer.mozilla.org/en-US/docs/Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F
Attachment #784702 -
Flags: review?(nfitzgerald) → review+
| Assignee | ||
Comment 4•12 years ago
|
||
Updated•12 years ago
|
Whiteboard: [land-in-fx-team]
Comment 5•12 years ago
|
||
Whiteboard: [land-in-fx-team] → [fixed-in-fx-team]
Comment 6•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 25
Updated•7 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•