Closed Bug 264915 Opened 15 years ago Closed 15 years ago

Remove FlushPendingEvents calls in nsEventStateManager

Categories

(Core :: Web Painting, defect, P1)

x86
Linux
defect

Tracking

()

RESOLVED FIXED
mozilla1.8alpha5

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

Details

These were added for bug 36849.  I suspect that given that we now batch view
updates and that visual style changes do a DEFERRED view update batch we can
eliminate this code altogether.  At the moment, we flush out all the reflows and
repaints on each mousemove...  That's really not called for.

I'll do some testing to make sure we don't regress bug 36849, and I'll wait till
after we're done with the regressions from bug 238493.  After that, I hope to be
able to remove a lot of this stuff.
Priority: -- → P1
Target Milestone: --- → mozilla1.8alpha5
Depends on: 244290
Hrm...  So I suspect that the event sequence described in bug 36849 comment 97
can still arise.  If so, we need to fix bug 244366 before we can fix this one. 
roc, did I understand the situation correctly?
Depends on: 244366
Yes, I think so. 244366 is definitely the right thing to do.
Blocks: 203448
Fixed by patch in bug 244366.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Component: Layout: View Rendering → Layout: Web Painting
You need to log in before you can comment on or make changes to this bug.