Closed Bug 874770 Opened 7 years ago Closed 7 years ago

Always enable OMTA throttling

Categories

(Core :: Graphics: Layers, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla24

People

(Reporter: dzbarsky, Assigned: dzbarsky)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Patch (obsolete) — Splinter Review
No description provided.
Attachment #752555 - Flags: review?(ncameron)
Comment on attachment 752555 [details] [diff] [review]
Patch

Review of attachment 752555 [details] [diff] [review]:
-----------------------------------------------------------------

I think we should check the OMTA pref instead of the throttled pref where you remove the check altogether. I can't persuade myself that we won't hit those blocks for main thread animation at the moment.
Attached patch PatchSplinter Review
You're right.  Also it may be worth merging these prefs at some point.
Attachment #752555 - Attachment is obsolete: true
Attachment #752555 - Flags: review?(ncameron)
Attachment #752621 - Flags: review?(ncameron)
Comment on attachment 752621 [details] [diff] [review]
Patch

Review of attachment 752621 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM
Attachment #752621 - Flags: review?(ncameron) → review+
https://hg.mozilla.org/mozilla-central/rev/0a801c3516c0
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
You need to log in before you can comment on or make changes to this bug.