unused parameter aFrameToUse in imgContainer::DoComposite

RESOLVED FIXED in mozilla10

Status

()

Core
ImageLib
RESOLVED FIXED
8 years ago
6 years ago

People

(Reporter: Joe Drew (not getting mail), Assigned: BenWa)

Tracking

Trunk
mozilla10
x86
Mac OS X
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

8 years ago
Created attachment 410600 [details] [diff] [review]
remove aFrameToUse

After bug 753 landed, we stopped needing to use the result of what DoComposite set aFrameToUse to, but never removed it as a parameter. This patch does that.
Attachment #410600 - Flags: review?(bobbyholley)
(Reporter)

Updated

8 years ago
Attachment #410600 - Attachment is patch: true
Attachment #410600 - Attachment mime type: application/octet-stream → text/plain
Comment on attachment 410600 [details] [diff] [review]
remove aFrameToUse

r+
Attachment #410600 - Flags: review?(bobbyholley) → review+
(Assignee)

Comment 2

6 years ago
Created attachment 565704 [details] [diff] [review]
remove aFrameToUse fix rot

Fix bitrot but still asking for review to make sure we still want this change.
Attachment #565704 - Flags: review?(bobbyholley+bmo)
Comment on attachment 565704 [details] [diff] [review]
remove aFrameToUse fix rot

it's an internal API, and the value is only ever written to. r+!
Attachment #565704 - Flags: review?(bobbyholley+bmo) → review+
(Assignee)

Comment 4

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/6e166aaa5be6
Assignee: joe → bgirard
Whiteboard: [inbound]
https://hg.mozilla.org/mozilla-central/rev/6e166aaa5be6
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla10
You need to log in before you can comment on or make changes to this bug.