Closed
Bug 1244901
Opened 6 years ago
Closed 6 years ago
Update potential checkerboarding duration probe to be entirely compositor-thread
Categories
(Core :: Panning and Zooming, defect)
Core
Panning and Zooming
Tracking
()
RESOLVED
FIXED
mozilla47
Tracking | Status | |
---|---|---|
firefox47 | --- | fixed |
People
(Reporter: kats, Assigned: kats)
References
Details
Attachments
(1 file)
The CHECKERBOARD_POTENTIAL_DURATION probe is computed by the PotentialCheckerboardDurationTracker class, which records checkerboard duration from the compositor thread and transform duration from the controller thread. For consistency and thread-safety, we should move the transform duration recording to the compositor thread as well. Conceptually this makes sense because checkerboard duration is recorded as time on the compositor thread, so the potential duration should also be time on the compositor thread.
Assignee | ||
Comment 1•6 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/33151/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/33151/
Attachment #8714575 -
Flags: review?(botond)
Updated•6 years ago
|
Attachment #8714575 -
Flags: review?(botond) → review+
Comment 2•6 years ago
|
||
Comment on attachment 8714575 [details] MozReview Request: Bug 1244901 - When recording transform duration for potential checkerboarding time, do so on the compositor thread. r?botond https://reviewboard.mozilla.org/r/33151/#review29899
Comment 4•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/bfae6347a978
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
You need to log in
before you can comment on or make changes to this bug.
Description
•