Closed Bug 653270 Opened 10 years ago Closed 10 years ago
SMILAnimation Controller construction
While looking at another bug, I noticed that nsSMILAnimationController construction/initialization is much more COM-ish and messy than it needs to be. Filing this bug on cleaning that up.
Here's the fix. Basically, NS_NewSMILAnimationController() is only ever called once, and it calls both the nsSMILAnimationController constructor and Init(). (and it's the only client of those) So I'm merging all of the above into the nsSMILAnimationController constructor.
Attachment #528725 - Flags: review?(birtles)
Attachment #528725 - Flags: review?(birtles) → review+
Target Milestone: --- → mozilla6
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment on attachment 528725 [details] [diff] [review] fix This needs to land on aurora in order for Bug 653238 and Bug 654015 to land. Very low-risk -- this just merges a few constructor-ish functions together for simplicity, with no functional change. (See summary in comment 1.) Requesting approval for both aurora and beta so this can land with Bug 653238 and Bug 654015.
(In reply to comment #3) > This needs to land on aurora in order for Bug 653238 and Bug 654015 to land. (s/aurora/aurora and beta/)
You need to log in before you can comment on or make changes to this bug.