Closed Bug 56128 Opened 25 years ago Closed 18 years ago

In nsObjectFrame::Paint, aDamageRect.width is zero if margins are zero

Categories

(Core :: Layout, defect, P4)

All
Windows NT
defect

Tracking

()

RESOLVED INCOMPLETE
Future

People

(Reporter: peterlubczynski-bugs, Assigned: peterlubczynski-bugs)

References

()

Details

(Keywords: perf, Whiteboard: [PL2:NA])

When you set the margins to zero on a page with plugins, when Paint is called for the Object Frame, the width of the damage rect is always passed in as zero.
Reassigning to Buster. Peter, if this is blocking rtm++ plugin bugs, please note that.
Assignee: clayton → buster
Currently, we don't optimize the painting region by intersecting with the damage rect because this damage rect isn't always correct. We just paint the whole plugin which causes it to flicker. The intersection is currently #ifdef'ed out and if this bug is something easy to fix, optimization can be turned back on.
per conversation with Peter, marking Future. This is primarily a perf bug. No pages that we know about suffer so much that they are unusable. Peter, 2 things: 1) what platform(s) is this? You have this marked platform=Macintosh with WinNT OS, and last I checked there weren't too many of those out there. 2) could you attach a test case that triggers the problem?
Status: NEW → ASSIGNED
Keywords: perf
Target Milestone: --- → Future
Reassigning QA Contact for all open and unverified bugs previously under Lorca's care to Gerardo as per phone conversation this morning.
QA Contact: lorca → gerardok
qa contact updated.
QA Contact: gerardok → bsharma
nominating for dogfood (from sdagley's list of bugs that are good candidates for our next release)
Keywords: nsdogfood
Blocks: 62891
Keywords: nsCatFood
Keywords: nsdogfood
SPAM. HTML Element component deprecated, changing component to Layout. See bug 88132 for details.
Component: HTML Element → Layout
buster is gone!
Assignee: buster → peterlubczynski
Status: ASSIGNED → NEW
QA Contact: bsharma → shrir
Priority: P3 → P4
Hardware: Macintosh → All
Whiteboard: [PL2:NA]
Peter, is this still an issue?
Ancient bug, no feedback. Closing.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.