Closed
Bug 1120050
Opened 9 years ago
Closed 9 years ago
Expose Skia scaler internals for use by downscale-during-decode
Categories
(Core :: Graphics, defect)
Core
Graphics
Tracking
()
RESOLVED
FIXED
mozilla38
People
(Reporter: seth, Assigned: seth)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
Downscale-during-decode needs to work in a streaming fashion, which the public Skia scaler API (the one we use for high-quality scaling) doesn't support. To implement downscale-during-decode, we need to expose some internal features of the Skia scaler API.
Assignee | ||
Comment 1•9 years ago
|
||
Here's the patch. This patch shouldn't change any functionality; it just moves internal Skia code that we need for downscale-during-decode into header files and otherwise makes it public. Purely refactoring.
Attachment #8546979 -
Flags: review?(tnikkel)
Comment 2•9 years ago
|
||
Comment on attachment 8546979 [details] [diff] [review] Expose Skia scaler internals for use by downscale-during-decode rubber stamp this. Jeff can do post landing review.
Attachment #8546979 -
Flags: review?(tnikkel)
Attachment #8546979 -
Flags: review?(jmuizelaar)
Attachment #8546979 -
Flags: review+
Assignee | ||
Comment 3•9 years ago
|
||
Pushed: remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/3ff6c190426e Based on this try job, which is looking pretty green: https://tbpl.mozilla.org/?tree=Try&rev=f3f4d04d1fbe
Comment 4•9 years ago
|
||
sorry had to back this out for bustage like https://treeherder.mozilla.org/logviewer.html#?job_id=5344852&repo=mozilla-inbound
Flags: needinfo?(seth)
Updated•9 years ago
|
Attachment #8546979 -
Flags: review?(jmuizelaar) → review+
Assignee | ||
Comment 6•9 years ago
|
||
This needed a rebase.
Assignee | ||
Updated•9 years ago
|
Attachment #8546979 -
Attachment is obsolete: true
Assignee | ||
Comment 7•9 years ago
|
||
Should be ready to go now. Pushed: https://hg.mozilla.org/integration/mozilla-inbound/rev/e00afa8044c3
Flags: needinfo?(seth)
Comment 8•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/e00afa8044c3
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
Comment 9•9 years ago
|
||
Comment on attachment 8550071 [details] [diff] [review] Expose Skia scaler internals for use by downscale-during-decode Review of attachment 8550071 [details] [diff] [review]: ----------------------------------------------------------------- ::: gfx/2d/image_operations.h @@ +183,5 @@ > + return 0.0f; // Outside of the window. > + if (x > -std::numeric_limits<float>::epsilon() && > + x < std::numeric_limits<float>::epsilon()) > + return 1.0f; // Special case the discontinuity at the origin. > + float xpi = x * static_cast<float>(M_PI); Missing #include "mozilla/Constants.h"
You need to log in
before you can comment on or make changes to this bug.
Description
•