Closed
Bug 115526
Opened 23 years ago
Closed 13 years ago
clipping and border drawing need ConditionRect
Categories
(Core :: Web Painting, defect, P1)
Tracking
()
RESOLVED
WORKSFORME
Future
People
(Reporter: dbaron, Assigned: dbaron)
References
(Blocks 1 open bug)
Details
(Whiteboard: [patch])
Attachments
(2 files, 3 obsolete files)
13.81 KB,
patch
|
Details | Diff | Splinter Review | |
775 bytes,
text/html
|
Details |
Clipping and border drawing need to pass through ConditionRect, at least on Linux. See attached testcase, which should show a really tall BODY element with border, background, and clipping in the same place. The background is right, but the border and clipping are wrong. Other platforms should be tested as well.
Assignee | ||
Comment 1•23 years ago
|
||
Assignee | ||
Updated•23 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•23 years ago
|
||
*** Bug 123146 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 3•23 years ago
|
||
The duplicate shows problems with dashed border drawing code as well.
Comment 4•23 years ago
|
||
*** Bug 128649 has been marked as a duplicate of this bug. ***
Assignee | ||
Updated•22 years ago
|
Target Milestone: --- → mozilla1.0
Comment 5•22 years ago
|
||
*** Bug 134533 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 6•22 years ago
|
||
I haven't tested this yet, and there's also an "XXX" comment about a bug I already know it has.
Assignee | ||
Comment 7•22 years ago
|
||
That's still not quite right, even when I changed the patch like this: 48c48 < @@ -1037,23 +1039,77 @@ --- > @@ -1037,23 +1039,80 @@ 105c105,107 < + // XXX What if we're out-of-bounds on a different side?! --- > + // XXX What if we're out-of-bounds on a different side?! (Note > + // also that ConditionPoint might divide by zero in such a > + // case.) 116a119 > + prevConditioned = PR_FALSE;
Assignee | ||
Updated•22 years ago
|
Target Milestone: mozilla1.0 → mozilla1.1beta
Assignee | ||
Updated•22 years ago
|
Target Milestone: mozilla1.1beta → mozilla1.2alpha
Comment 8•22 years ago
|
||
*** Bug 167936 has been marked as a duplicate of this bug. ***
Assignee | ||
Updated•22 years ago
|
Severity: normal → major
Priority: -- → P1
Target Milestone: mozilla1.2alpha → Future
Assignee | ||
Comment 9•22 years ago
|
||
*** Bug 171917 has been marked as a duplicate of this bug. ***
Assignee | ||
Updated•22 years ago
|
Whiteboard: [patch]
Assignee | ||
Comment 10•22 years ago
|
||
This seems to work, except for some invalidation problems on borders that may or may not be in this area of the code. (They probably are, though.)
Attachment #78151 -
Attachment is obsolete: true
Assignee | ||
Comment 11•22 years ago
|
||
*** Bug 188675 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 12•21 years ago
|
||
The patch for bug 86249 will probably fix many of the border-related symptoms of this bug.
Comment 13•21 years ago
|
||
The testcase looks worse in Firefox 0.8 and Mozilla 1.6 than in Mozilla 1.5. I don't know how the clipped part is supposed to look, but in Mozilla 1.5 the yellow area is nicely contained within the green frame all the way down. In Firefox 0.8 and Mozilla 1.6 however, the yellow spills over the bottom green border.
Comment 14•20 years ago
|
||
There is still a problem with the yellow overflowing the bottom of the rectangle. Seen in Mozilla 1.8a3/Linux and Netscape 7.2/Linux.
Comment 15•20 years ago
|
||
Comment #14 applies to 1.8a5 as well. Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8a5) Gecko/20041121
Comment 16•20 years ago
|
||
I see no improvement in 1.8a6. Given that this bug has the highest Priority (P1), has major Severity, and blocks a number of other bugs, I think it ought to be fixed in Mozilla 1.8. Thus, I'm nominating this bug to block the 1.8b release.
Flags: blocking1.8b?
Assignee | ||
Comment 17•20 years ago
|
||
too late for 1.8b, transferring nomination to 1.8b2
Flags: blocking1.8b?
Flags: blocking1.8b2?
Flags: blocking1.8b-
Comment 18•19 years ago
|
||
For the record: the testcase shows no visible improvement in Mozilla 1.8b.
Assignee | ||
Updated•19 years ago
|
Component: Layout → GFX: Gtk
Assignee | ||
Comment 19•19 years ago
|
||
The last five comments have all been of no help whatsoever. Please stop adding non-technical comments to this bug. If you continue, I won't be able to find the actually useful comments within them and will be less likely to fix the bug.
Assignee | ||
Comment 20•19 years ago
|
||
Attachment #101494 -
Attachment is obsolete: true
Assignee | ||
Comment 21•19 years ago
|
||
This doesn't seem to fix bug 180877, bug 252241, or bug 290473.
We probably only want ConditionRect to be used on GTK1. GTK2 should already be OK, in theory. You probably want a typedef for nsAutoBuffer<GdkPoint,8>.
Comment 23•19 years ago
|
||
too late for beta2 - try for next milestone
Flags: blocking1.8b3?
Flags: blocking1.8b2?
Flags: blocking1.8b2-
Updated•19 years ago
|
Flags: blocking1.8b3? → blocking1.8b3-
Comment 24•17 years ago
|
||
David, now that gfx/src/gtk/ has been removed on trunk, is this bug/patch still relevant for trunk?
Updated•16 years ago
|
Product: Core → Core Graveyard
Comment 25•13 years ago
|
||
I am not clear on what the correct rendering of the test case ought to have been, but since we no longer support Gtk1, I'm inclined to think this is no longer relevant. But I'm not going to yank a bug out from under dbaron. :) It can moulder in the proper component till he gets around to it again.
Component: GFX: Gtk → Layout: View Rendering
Product: Core Graveyard → Core
QA Contact: chrispetersen → layout.view-rendering
Assignee | ||
Comment 26•13 years ago
|
||
attachment 61907 [details] updated for changes to CSS 'overflow' propagation rules for HTML and BODY.
Attachment #61907 -
Attachment is obsolete: true
Assignee | ||
Comment 27•13 years ago
|
||
Yeah, I no longer see any problems with the testcase (though I recall we still did have problems in the early cairo days).
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → WORKSFORME
Comment 28•13 years ago
|
||
Do you think it's worth checking in a reftest?
Updated•6 years ago
|
Component: Layout: View Rendering → Layout: Web Painting
You need to log in
before you can comment on or make changes to this bug.
Description
•