marker leaves poopies along the 4 cardinal directions (on my machine anyway)

RESOLVED FIXED

Status

()

Core
SVG
RESOLVED FIXED
12 years ago
11 years ago

People

(Reporter: Bill Burdick, Assigned: tor)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

12 years ago
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8b5) Gecko/20051006 Firefox/1.4.1
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8b5) Gecko/20051006 Firefox/1.4.1

When I drag around the red circle, the arrow head leaves mouse poop on the screen.

Reproducible: Always

Steps to Reproduce:
1.open url
2.drag red circle around green circle
3.there is no step 3

Actual Results:  
poopies

Expected Results:  
no poopies
Confirming. This also happens on w32, for both GDI+ and cairo. It seems were not
repainting the correct area somehow.
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Linux → All
Hardware: PC → All
(Reporter)

Comment 2

12 years ago
Created attachment 199421 [details]
testcase that shows SVG marker poopies
(Assignee)

Comment 3

12 years ago
Created attachment 199423 [details] [diff] [review]
do full invalidate logic on redraw unsuspend
Assignee: general → tor
Status: NEW → ASSIGNED
Attachment #199423 - Flags: review?(scootermorris)

Comment 4

12 years ago
Comment on attachment 199423 [details] [diff] [review]
do full invalidate logic on redraw unsuspend

Looks fine to me.  I haven't tested to see what the performance implications
might be, though.
Attachment #199423 - Flags: review?(scootermorris) → review+
(Assignee)

Comment 5

12 years ago
Might be a bit slower in the case of filtered and/or marked path geometry, but
it's the right thing to do.

Checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
[got here from bug 339220] What now does the setting of mUpdateFlags to 0?  Also, why use 0 for a PRBool when you should be using PR_FALSE?
Er, for the PRBool issue, I was looking at the trunk when I shouldn't have been.
You need to log in before you can comment on or make changes to this bug.