Closed Bug 848841 Opened 7 years ago Closed 7 years ago

Fix bustage from Bug 846132 (Remove imgIContainer::CopyFrame)

Categories

(SeaMonkey :: OS Integration, defect)

x86
Windows 7
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.19

People

(Reporter: philip.chee, Assigned: philip.chee)

References

Details

Attachments

(1 file)

I've got a working patch.
Assignee: nobody → philip.chee
Status: NEW → ASSIGNED
Attachment #722331 - Flags: review?(neil)
Comment on attachment 722331 [details] [diff] [review]
Patch 1.0 straight copy.

>-  nsresult rv = aImage->CopyFrame(imgIContainer::FRAME_CURRENT,
...
>+  aImage->GetFrame(imgIContainer::FRAME_FIRST,
r=me if you keep FRAME_CURRENT.
Attachment #722331 - Flags: review?(neil) → review+
>>+  aImage->GetFrame(imgIContainer::FRAME_FIRST,
> r=me if you keep FRAME_CURRENT.

Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/f41f0894859a
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.19
Thanks for fixing this, Philip! I'm puzzled that this didn't show up on try for me.
> Thanks for fixing this, Philip! I'm puzzled that this didn't show up on try for me
That's because this is a comm-central patch. And obviously you'd need to use the comm-central try servers ;)
Heh, that would explain it. =)
You need to log in before you can comment on or make changes to this bug.