Bug 1640969 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
Markers currently [have a `time` field in the output JSON[(https://searchfox.org/mozilla-central/rev/7dafc35406b9c945189c617d427f5458933fd3fb/tools/profiler/core/ProfiledThreadData.cpp#244). Many markers in the payloads also include a `startTime` and `endTime`. These various fields end up being ambiguous in how they are used.

For the output marker table we should remove the `time` field, and add `startTime` and `endTime`. Each of these properties can then be `null` or `number`.

This will also need a profiler upgrader for the front-end which will need to land first.
Revision 1 by
on 
Markers currently [have a `time` field in the output JSON](https://searchfox.org/mozilla-central/rev/7dafc35406b9c945189c617d427f5458933fd3fb/tools/profiler/core/ProfiledThreadData.cpp#244). Many markers in the payloads also include a `startTime` and `endTime`. These various fields end up being ambiguous in how they are used.

For the output marker table we should remove the `time` field, and add `startTime` and `endTime`. Each of these properties can then be `null` or `number`.

This will also need a profiler upgrader for the front-end which will need to land first.

Back to Bug 1640969 Comment 0