Closed Bug 1517980 Opened 6 years ago Closed 6 years ago

Have CONTENT_FRAME_TIME_REASON correspond to CONTENT_FRAME_TIME_VSYNC

Categories

(Core :: Graphics: WebRender, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla66
Tracking Status
firefox66 --- fixed

People

(Reporter: jrmuizel, Assigned: jrmuizel)

Details

Attachments

(1 file)

CONTENT_FRAME_TIME_VSYNC is more meaningful metric because it actually starts at the beginning of the vsync making it easier to reason about missing frames. It makes more sense for CONTENT_FRAME_TIME_REASON wants to use this same starting point.
CONTENT_FRAME_TIME_VSYNC is more meaningful metric because it actually starts at the beginning of the vsync making it easier to reason about missing frames. It makes more sense for CONTENT_FRAME_TIME_REASON wants to use this same starting point.
Pushed by jmuizelaar@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f55412c54286 Have CONTENT_FRAME_TIME_REASON correspond to CONTENT_FRAME_TIME_VSYNC. r=mattwoodrow
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
Assignee: nobody → jmuizelaar
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: