Closed
Bug 456152
Opened 16 years ago
Closed 16 years ago
slight API cleanup for nsCSSRendering::PaintBackground
Categories
(Core :: Web Painting, defect)
Core
Web Painting
Tracking
()
RESOLVED
FIXED
mozilla1.9.1b1
People
(Reporter: zwol, Assigned: zwol)
Details
Attachments
(1 file)
9.63 KB,
patch
|
roc
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
Almost all callers of nsCSSRendering::PaintBackground pass the padding and border style structs associated with the frame they're also passing, having just looked them up in order to do so. The sole exception can safely be changed to use ::PaintBackgroundWithSC instead. So this patch removes those arguments. guessing at reviewer; feel free to redirect.
Attachment #339540 -
Flags: review?(roc)
Comment on attachment 339540 [details] [diff] [review] (rev 0) patch [Checkin: Comment 2] great, thanks
Attachment #339540 -
Flags: superreview+
Attachment #339540 -
Flags: review?(roc)
Attachment #339540 -
Flags: review+
Assignee | ||
Updated•16 years ago
|
Keywords: checkin-needed
Comment 2•16 years ago
|
||
Comment on attachment 339540 [details] [diff] [review] (rev 0) patch [Checkin: Comment 2] http://hg.mozilla.org/mozilla-central/rev/e8eaf2c11d9f
Attachment #339540 -
Attachment description: (rev 0) patch → (rev 0) patch
[Checkin: Comment 2]
Updated•16 years ago
|
Severity: normal → minor
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1b1
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
•