Closed Bug 444692 Opened 16 years ago Closed 16 years ago

Plugin invalidation should invalidate via frames rather than views

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: roc, Assigned: roc)

References

Details

Attachments

(1 file)

Attached patch fix (deleted) — Splinter Review
We want to add hooks to nsFrame::InvalidateInternal so that the invalidation area can be transformed or the damage propagated to other elements that are observing the frame's rendering. But currently windowless plugins invalidate themselves directly via the view. Instead we should invalidate via the frame. This is also simpler code.
Attachment #329013 - Flags: superreview?(dbaron)
Attachment #329013 - Flags: review?(dbaron)
Comment on attachment 329013 [details] [diff] [review] fix r+sr=dbaron
Attachment #329013 - Flags: superreview?(dbaron)
Attachment #329013 - Flags: superreview+
Attachment #329013 - Flags: review?(dbaron)
Attachment #329013 - Flags: review+
Pushed 1e22446f1b6f.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: