Closed Bug 1435248 Opened 2 years ago Closed 2 years ago

Clean up some compositor-scheduling stuff in CompositorBridgeParent and nearby code

Categories

(Core :: Graphics: Layers, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla60
Tracking Status
firefox60 --- fixed

People

(Reporter: kats, Assigned: kats)

Details

Attachments

(3 files)

Following from bug 1435022 I have some more cleanups outside CompositorVsyncScheduler.
Comment on attachment 8947843 [details]
Bug 1435248 - Use a Maybe<TimeStamp> instead of a bool+TimeStamp.

https://reviewboard.mozilla.org/r/217528/#review224082
Attachment #8947843 - Flags: review?(nical.bugzilla) → review+
Comment on attachment 8947844 [details]
Bug 1435248 - Drop unnecessary virtualization of function.

https://reviewboard.mozilla.org/r/217530/#review224084
Attachment #8947844 - Flags: review?(nical.bugzilla) → review+
Comment on attachment 8947845 [details]
Bug 1435248 - Fix function naming.

https://reviewboard.mozilla.org/r/217532/#review224086
Attachment #8947845 - Flags: review?(nical.bugzilla) → review+
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/54134ad6f663
Use a Maybe<TimeStamp> instead of a bool+TimeStamp. r=nical
https://hg.mozilla.org/integration/autoland/rev/8482ce1b16bb
Drop unnecessary virtualization of function. r=nical
https://hg.mozilla.org/integration/autoland/rev/c7a373f1fd69
Fix function naming. r=nical
You need to log in before you can comment on or make changes to this bug.