Closed Bug 526596 Opened 16 years ago Closed 16 years ago

CSS outline is not coalesced in dynamic case

Categories

(Core :: Layout, defect)

x86
macOS
defect
Not set
minor

Tracking

()

RESOLVED FIXED

People

(Reporter: jruderman, Assigned: bzbarsky)

References

Details

(Keywords: testcase)

Attachments

(3 files, 1 obsolete file)

Attached file testcase (dynamic)
Based on layout/reftests/bugs/424236-5.html. Probably a regression.
Attached file reference
Yes. The key part is that the two testcases have different frame trees. I guess I can actually make this case (the trailing inline is empty and a block is the first thing appended) generate the same frame tree as the static case. Let me give that a shot.
Hmm. So here's a question about the continuation model. Is it possible to have an inline whose first continuation has no kids but which has some other continuation that does have kids?
Though I suspect that if that answer is "yes" some of our other append code might not be quite correct... so I hope it's actually "no".
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #413782 - Flags: review?(roc)
Attachment #413782 - Attachment is obsolete: true
Attachment #413791 - Flags: review?(roc)
Attachment #413782 - Flags: review?(roc)
Though I guess the other approach might be faster if we have lots of continuations for that trailing inline..... whichever one you like more?
Why didn't the outline merging code detect this case, anyway? Should it have? (Should we be fixing this bug in two different ways?)
Possibly. How should outline merging generally work for ib splits? Where do we merge outlines, in general?
I suppose the little unioning we do (which isn't nearly what we should) is in nsCSSRendering::PaintOutline, about 20 lines in.
Ah. That's not actual unioning; that's just drawing the outline around a different rect. It seems to me that we don't actually coalesce outlines in a sane way at all. Consider: data:text/html,<!DOCTYPE html><span style="outline: 5px solid black">aaa<br>bbb</span>
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Depends on: 535721
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: