z-index changes should not trigger reflow

RESOLVED FIXED in mozilla1.9.3a1

Status

()

RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: bzbarsky, Assigned: bzbarsky)

Tracking

Trunk
mozilla1.9.3a1
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

nsStylePosition::CalcDifference returns a reflow hint for z-index changes.  This should probably need no more than a SyncFrameView hint, or RepaintFrame hint.  See bug 502288 comment 2.
(Assignee)

Comment 1

9 years ago
Oh, and maybe we should do clip here too...  that ought not need to reflow either.
We don't really need SyncFrameView anymore, view z-indexes don't really do anything. They used to sort of affect widget ordering but that's irrelevant now, certainly for Web content.
(Assignee)

Comment 3

9 years ago
So this should just be a RepaintFrame, right?
(Assignee)

Comment 5

9 years ago
Created attachment 408408 [details] [diff] [review]
Fix
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #408408 - Flags: review?(roc)
(Assignee)

Comment 6

9 years ago
Push http://hg.mozilla.org/mozilla-central/rev/44b6cab55e0a
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a1
You need to log in before you can comment on or make changes to this bug.