Closed Bug 825809 Opened 8 years ago Closed 8 years ago
Waiting For Content To Paint should be true after m Paint Throttler .Task Complete post new task
I've found one problem with mWaitingForContentToPaint and http://mxr.mozilla.org/mozilla-central/source/gfx/layers/ipc/AsyncPanZoomController.cpp#1156 Basically we have case when RequestContentRepaint posted but not tracked by mWaitingForContentToPaint. this happen when we call mPaintThrottler.TaskComplete which Run queued task, and do not set mWaitingForContentToPaint = true in that case.
Assignee: nobody → romaxa
Status: NEW → ASSIGNED
Attachment #696928 - Flags: review?(jones.chris.g)
Btw, this patch fixes content shaking during scroll with FLING
Attachment #696928 - Flags: review?(jones.chris.g) → review?(ajones)
8 years ago
Attachment #696928 - Flags: review?(ajones) → review+
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
We'll track, please nominate for approval with reasons for uplift and risk assessment.
Comment on attachment 696928 [details] [diff] [review] Track waiting for task executed in TaskComplete [Approval Request Comment] Bug caused by (feature/regressing bug #): 799401 User impact if declined: Unpainted (white) regions on screen. Testing completed: Has been on m-c for some time. Risk to taking this patch (and alternatives if risky): Low. The bug was a clear oversight that missed an edge case. String or UUID changes made by this patch: None.
Attachment #696928 - Flags: approval-mozilla-b2g18?
Attachment #696928 - Flags: approval-mozilla-b2g18? → approval-mozilla-b2g18+
You need to log in before you can comment on or make changes to this bug.