The default bug view has changed. See this FAQ.

Rename aEffectsFrame to just aFrame in nsSVGIntegrationUtils

RESOLVED FIXED in mozilla16

Status

()

Core
SVG
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: jwatt, Assigned: jwatt)

Tracking

Trunk
mozilla16
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

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

5 years ago
Created attachment 637493 [details] [diff] [review]
patch
Attachment #637493 - Flags: review?(longsonr)
Comment on attachment 637493 [details] [diff] [review]
patch

Stealing review.
Attachment #637493 - Flags: review?(longsonr) → review+
(Assignee)

Comment 3

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/10ae67e2c60a
Target Milestone: --- → mozilla16
https://hg.mozilla.org/mozilla-central/rev/10ae67e2c60a
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.