Closed
Bug 527325
Opened 15 years ago
Closed 15 years ago
SVG feConvoleMatrix doesn't process bias correctly
Categories
(Core :: SVG, defect)
Core
SVG
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
status1.9.2 | --- | beta3-fixed |
People
(Reporter: krit, Assigned: longsonr)
Details
Attachments
(2 files, 1 obsolete file)
361 bytes,
image/svg+xml
|
Details | |
3.17 KB,
patch
|
roc
:
review+
roc
:
approval1.9.2+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; de; rv:1.9.1.4) Gecko/20091028 Ubuntu/9.10 (karmic) Firefox/3.5.4 Build Identifier: There is no bias support for feConvolveMatrix. Reproducible: Always
Reporter | ||
Comment 1•15 years ago
|
||
example for bias. You should see a light green box.
Assignee | ||
Updated•15 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: SVG feConvoleMatrix doesn't support bias → SVG feConvoleMatrix doesn't process bias correctly
Assignee | ||
Updated•15 years ago
|
OS: Linux → All
Hardware: x86 → All
Assignee | ||
Comment 2•15 years ago
|
||
Assignee: nobody → longsonr
Assignee | ||
Comment 3•15 years ago
|
||
Comment on attachment 411098 [details] [diff] [review] patch wrong.
Attachment #411098 -
Attachment is obsolete: true
Assignee | ||
Comment 4•15 years ago
|
||
We multiply bias by 255 at the beginning of the ConvolvePixel method and again when we use it. Once is enough.
Attachment #411287 -
Flags: review?(roc)
Attachment #411287 -
Flags: review?(roc) → review+
Assignee | ||
Comment 5•15 years ago
|
||
pushed http://hg.mozilla.org/mozilla-central/rev/12b789f71d70
Status: NEW → RESOLVED
Closed: 15 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Assignee | ||
Updated•15 years ago
|
Attachment #411287 -
Flags: approval1.9.2?
Assignee | ||
Comment 6•15 years ago
|
||
Simple low risk fix with test.
Attachment #411287 -
Flags: approval1.9.2? → approval1.9.2+
Assignee | ||
Updated•15 years ago
|
status1.9.2:
--- → final-fixed
Assignee | ||
Comment 7•15 years ago
|
||
pushed http://hg.mozilla.org/releases/mozilla-1.9.2/rev/9b504577f938
You need to log in
before you can comment on or make changes to this bug.
Description
•