Closed Bug 629838 Opened 9 years ago Closed 9 years ago

Fire MozAfterPaint events when empty transactions occur

Categories

(Core :: Layout, defect)

x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla2.0b12
Tracking Status
blocking2.0 --- final+

People

(Reporter: roc, Assigned: kael)

References

Details

(Keywords: regression, Whiteboard: [softblocker])

Attachments

(1 file, 1 obsolete file)

Like bug 625248 but for MozAfterPaint events. This should be easier, you just need to call NotifyDidPaintForSubtree in the right place.

You'll need to write tests though.
blocking2.0: --- → final+
Whiteboard: [softblocker]
I cannot come up with a reproduction case where trunk fails to fire MozAfterPaint for empty transactions, though I admit I don't understand the graphics stack very well at this point. Can you provide a repro, or at least a description of one?
I've attached a patch that dispatches the MozAfterPaint event after incrementing mozPaintCount during an empty transaction. The patch includes a couple tests based on the ones for the mozPaintCount bug.

Note that this patch won't merge correctly unless my patch from #625248 has been applied.
Attachment #508644 - Flags: review?(roc)
Depends on: 625248
Looks great!

+        frame->PresContext()->NotifyDidPaintForSubtree();

Just use 'presContext'
Patch updated.
Attachment #508644 - Attachment is obsolete: true
Attachment #508855 - Flags: review?(roc)
Attachment #508644 - Flags: review?(roc)
Attachment #508855 - Flags: review?(roc) → review+
Keywords: checkin-needed
Whiteboard: [softblocker] → [softblocker][needs landing]
http://hg.mozilla.org/mozilla-central/rev/f4c949162937
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: in-testsuite+
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [softblocker][needs landing] → [softblocker]
Target Milestone: --- → mozilla2.0b12
I backed this patch as part of this pushlog <http://hg.mozilla.org/mozilla-central/pushloghtml?changeset=7e12e3e16e6c> because of the oranges it caused <http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1296852850.1296854923.2345.gz&fulltext=1>.

I'm not sure which one of the bugs was at fault, so I just backed them all out.  The assignee needs to investigate and make sure that his patch has not been the culprit, and then re-add checkin-needed.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(In reply to comment #6)
> I backed this patch as part of this pushlog
> <http://hg.mozilla.org/mozilla-central/pushloghtml?changeset=7e12e3e16e6c>
> because of the oranges it caused
> <http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1296852850.1296854923.2345.gz&fulltext=1>.
> 
> I'm not sure which one of the bugs was at fault, so I just backed them all out.
>  The assignee needs to investigate and make sure that his patch has not been
> the culprit, and then re-add checkin-needed.

Sorry, this belonged to bug 631329.
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → FIXED
Depends on: 631329
Relanded all but one of the patches in that push.  For this bug, that's:
http://hg.mozilla.org/mozilla-central/rev/10ecb300665e
The last four comments apparently all belonged on bug 631329.
You need to log in before you can comment on or make changes to this bug.