Closed
Bug 848841
Opened 12 years ago
Closed 12 years ago
Fix bustage from Bug 846132 (Remove imgIContainer::CopyFrame)
Categories
(SeaMonkey :: OS Integration, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.19
People
(Reporter: philip.chee, Assigned: philip.chee)
References
Details
Attachments
(1 file)
|
1.44 KB,
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
I've got a working patch.
|
Assignee | |
Comment 1•12 years ago
|
||
|
||
Comment 2•12 years ago
|
||
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+
|
|
Assignee | |
Comment 3•12 years ago
|
||
>>+ 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: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.19
|
||
Comment 4•12 years ago
|
||
Thanks for fixing this, Philip! I'm puzzled that this didn't show up on try for me.
|
|
Assignee | |
Comment 5•12 years ago
|
||
> 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 ;)
|
|
||
Comment 6•12 years ago
|
||
Heh, that would explain it. =)
You need to log in
before you can comment on or make changes to this bug.
Description
•