Closed
Bug 1275750
Opened 8 years ago
Closed 8 years ago
Remove some dead code in imgFrame
Categories
(Core :: Graphics: ImageLib, defect)
Core
Graphics: ImageLib
Tracking
()
RESOLVED
FIXED
mozilla49
Tracking | Status | |
---|---|---|
firefox49 | --- | fixed |
People
(Reporter: seth, Assigned: seth)
Details
Attachments
(3 files)
2.74 KB,
patch
|
eflores
:
review+
|
Details | Diff | Splinter Review |
3.78 KB,
patch
|
eflores
:
review+
|
Details | Diff | Splinter Review |
1.48 KB,
patch
|
eflores
:
review+
|
Details | Diff | Splinter Review |
There is some dead code laying around in imgFrame. Let's get rid of it.
Assignee | ||
Comment 1•8 years ago
|
||
ScalingData isn't used anywhere, because the old HQ scaling code has been completely removed and replaced with downscale-during-decode.
Attachment #8756612 -
Flags: review?(edwin)
Assignee | ||
Comment 2•8 years ago
|
||
GetDrawTarget() isn't used anywhere; VectorImage used to use it, but it now uses imgFrame::InitWithDrawable(). GetStride() can also be removed since only GetDrawTarget() uses it.
Attachment #8756613 -
Flags: review?(edwin)
Assignee | ||
Comment 3•8 years ago
|
||
Nothing uses GetFormat().
Attachment #8756614 -
Flags: review?(edwin)
Comment on attachment 8756612 [details] [diff] [review] (Part 1) - Remove unused ScalingData-related code. Review of attachment 8756612 [details] [diff] [review]: ----------------------------------------------------------------- yay
Attachment #8756612 -
Flags: review?(edwin) → review+
Comment on attachment 8756613 [details] [diff] [review] (Part 2) - Remove unused GetDrawTarget()-related code. Review of attachment 8756613 [details] [diff] [review]: ----------------------------------------------------------------- woo
Attachment #8756613 -
Flags: review?(edwin) → review+
Comment on attachment 8756614 [details] [diff] [review] (Part 3) - Remove imgFrame::GetFormat(). Review of attachment 8756614 [details] [diff] [review]: ----------------------------------------------------------------- <3 removing code
Attachment #8756614 -
Flags: review?(edwin) → review+
Assignee | ||
Comment 7•8 years ago
|
||
Thanks for the reviews, Edwin!
Assignee | ||
Comment 8•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/09070d5abf9f80a20fa9bb8d75c8ba3a253aec13 Bug 1275750 (Part 1) - Remove unused ScalingData-related code. r=edwin https://hg.mozilla.org/integration/mozilla-inbound/rev/7c4ec7324278eabb7a00fcd9c11191fd699feb40 Bug 1275750 (Part 2) - Remove unused GetDrawTarget()-related code. r=edwin https://hg.mozilla.org/integration/mozilla-inbound/rev/945c510ffa9b4af93cb438331af18ac82f4db59c Bug 1275750 (Part 3) - Remove imgFrame::GetFormat(). r=edwin
Comment 9•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/09070d5abf9f https://hg.mozilla.org/mozilla-central/rev/7c4ec7324278 https://hg.mozilla.org/mozilla-central/rev/945c510ffa9b
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox49:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in
before you can comment on or make changes to this bug.
Description
•