only schedule paint when apz aware event listener is added if event regions/apz are enabled

RESOLVED FIXED in Firefox 46

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: tnikkel, Assigned: tnikkel)

Tracking

unspecified
mozilla47
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox45 unaffected, firefox46 fixed, firefox47 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

3 years ago
Bug 1210057 added this, we should only do it if apz/event regions are enabled.
Assignee

Comment 1

3 years ago
Posted patch patchSplinter Review
Attachment #8725905 - Flags: review?(bugmail.mozilla)
Attachment #8725905 - Flags: review?(bugmail.mozilla) → review+

Comment 3

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/ff7425016da8
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Assignee

Comment 4

3 years ago
Comment on attachment 8725905 [details] [diff] [review]
patch

Followup to bug 1210057, since that is getting uplifted we should uplift this.
Approval Request Comment
[Feature/regressing bug #]: bug 1210057
[User impact if declined]: extra paints when apz-disabled
[Describe test coverage new/current, TreeHerder]: none
[Risks and why]: very safe, limits an apz-only change to be apz-only
[String/UUID change made/needed]: none
Attachment #8725905 - Flags: approval-mozilla-aurora?
Comment on attachment 8725905 [details] [diff] [review]
patch

Adds a limit for some APZ changes, please uplift to 46.
Attachment #8725905 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.