Closed
Bug 769238
Opened 12 years ago
Closed 12 years ago
Rename aEffectsFrame to just aFrame in nsSVGIntegrationUtils
Categories
(Core :: SVG, defect)
Core
SVG
Tracking
()
RESOLVED
FIXED
mozilla16
People
(Reporter: jwatt, Assigned: jwatt)
Details
Attachments
(1 file)
|
6.85 KB,
patch
|
dholbert
:
review+
|
Details | Diff | Splinter Review |
The name aEffectsFrame in nsSVGIntegrationUtils sounds like it's the frame providing the effects, rather than the frame to which effects are being applied. We could call it something more meaningful, but frankly the name aFrame seems fine in its context.
|
Assignee | |
Comment 1•12 years ago
|
||
Attachment #637493 -
Flags: review?(longsonr)
|
||
Comment 2•12 years ago
|
||
Comment on attachment 637493 [details] [diff] [review] patch Stealing review.
Attachment #637493 -
Flags: review?(longsonr) → review+
|
|
Assignee | |
Comment 3•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/10ae67e2c60a
Target Milestone: --- → mozilla16
|
||
Comment 4•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/10ae67e2c60a
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•