1445570 - Ensure notifying the MozAfterPaint correctly without timer.

Status: RESOLVED FIXED

(Core :: Layout, enhancement, P1)

Description

[Mantaroh Yoshinaga[:mantaroh]]

The bug 1419226 was going to remove the timer of MozAfterPaint. However, it affects many reftest and mochitest and so on, so bug 1419226 will land related test only. This bug tracks the removing the timer of MozAfterPaint.

Comment 1

[Brian Birtles (:birtles)]

Mantaroh, can you please attach the relevant patches from bug 1419226 for posterity? Just the patch files as attachments will do, no need to use MozReview. Thanks.

Comment 2

[Mantaroh Yoshinaga[:mantaroh]]

Attachment: work-in-progress-removing-timer.patch

(In reply to Brian Birtles (:birtles) from comment #1)
> Mantaroh, can you please attach the relevant patches from bug 1419226 for
> posterity? Just the patch files as attachments will do, no need to use
> MozReview. Thanks.

OK. This patch is removing timer of MozAfterPaint patch.
After applying this patch, several test are fail. For example, reftest of webrender and intermittents.
WebRender's reftest failure will avoid by removing initial browser tab's blank attribute[1].
A failure of reftest intermittent will avoid by calling nsDOMWindowUtil::UpdateLayerTree() after finishing MozAfterPaint listener handler[2].

[1] https://hg.mozilla.org/try/rev/c6a12e9ea6fa82ca72243a5af15894f4897763ef
[2] https://hg.mozilla.org/try/rev/7ba26a37f99f578191159534dfba43a64965db06

Comment 3

[Matt Woodrow (:mattwoodrow)]

Attachment: Bug 1445570 - Remove EnsureEventualAfterPaint timer. r?tnikkel

MozReview-Commit-ID: C7WICJ5Q0ES

Comment 4

[Timothy Nikkel (:tnikkel)]

Comment on attachment 9006441 [details]
Bug 1445570 - Remove EnsureEventualAfterPaint timer. r?tnikkel

Timothy Nikkel (:tnikkel) has approved the revision.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6fefaf842b05
Remove EnsureEventualAfterPaint timer. r=tnikkel

Comment 6

[Eliza Balazs [:ebalazs_]]

https://hg.mozilla.org/mozilla-central/rev/6fefaf842b05