Closed
Bug 382092
Opened 18 years ago
Closed 18 years ago
Simple canvas cleanups
Categories
(Core :: Graphics: Canvas2D, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: sharparrow1, Unassigned)
Details
Attachments
(1 file)
(deleted),
patch
|
vlad
:
review+
vlad
:
superreview+
|
Details | Diff | Splinter Review |
Some easy cleanups; the only non-trivial change is switching to using the newly fixed RenderDocument for doing drawWindow calls. Should make things easier to understand when you switch it to Thebes.
This does get rid of the branch compatibility of this file, but I don't think we care because our next release will be off 1.9 anyway.
Attachment #266174 -
Flags: review?(vladimir)
Reporter | ||
Comment 1•18 years ago
|
||
Should I ask someone else for review?
Comment on attachment 266174 [details] [diff] [review]
Patch
Sorry, I missed the review request.. I guess this is ok -- for a while there I was thinking to keep the MOZILLA_1_8 stuff around to keep this file in sync with the 1.8 branch version, in case I needed to fix anything on both. But that was a long time ago, so r+ :)
Attachment #266174 -
Flags: review?(vladimir) → review+
Reporter | ||
Comment 3•18 years ago
|
||
Comment on attachment 266174 [details] [diff] [review]
Patch
Could you make that an r+sr? (It doesn't seem worth bothering someone else for an sr for a patch like this.)
Attachment #266174 -
Flags: superreview?(vladimir)
Comment on attachment 266174 [details] [diff] [review]
Patch
whoops, yep, sorry
Attachment #266174 -
Flags: superreview?(vladimir) → superreview+
Reporter | ||
Comment 5•18 years ago
|
||
Checked in. This caused some unexpected passes on mac by fixing some rounding errors; my best guess is that it is due to using gfxASurface::CONTENT_COLOR as a parameter for aThebesContext->PushGroup instead of gfxASurface::CONTENT_COLOR_ALPHA. (Bugs affected: bug 382039, bug 382399, bug 382040, bug 381831, bug 381834, bug 383386.)
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Updated•18 years ago
|
Flags: in-testsuite-
You need to log in
before you can comment on or make changes to this bug.
Description
•