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

RESOLVED INCOMPLETE

Status

()

Core
Layout
P4
normal
RESOLVED INCOMPLETE
18 years ago
11 years ago

People

(Reporter: Peter Lubczynski, Assigned: Peter Lubczynski)

Tracking

({perf})

Trunk
Future
All
Windows NT
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [PL2:NA], URL)

(Assignee)

Description

18 years ago
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.

Comment 1

18 years ago
Reassigning to Buster. Peter, if this is blocking rtm++ plugin bugs, please 
note that.
Assignee: clayton → buster
(Assignee)

Comment 2

18 years ago
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.

Comment 3

18 years ago
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

Comment 5

17 years ago
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
(Assignee)

Updated

17 years ago
Blocks: 62891
Keywords: nsCatFood
Keywords: nsdogfood
SPAM. HTML Element component deprecated, changing component to Layout. See bug
88132 for details.
Component: HTML Element → Layout
(Assignee)

Comment 8

17 years ago
buster is gone!
Assignee: buster → peterlubczynski
Status: ASSIGNED → NEW
QA Contact: bsharma → shrir

Updated

16 years ago
Priority: P3 → P4
Hardware: Macintosh → All
Whiteboard: [PL2:NA]
Peter, is this still an issue?
Ancient bug, no feedback. Closing.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.