Closed
Bug 739969
Opened 13 years ago
Closed 13 years ago
Regression: Fixedpoint division in blur code was replaced to integer division
Categories
(Core :: Graphics, defect)
Core
Graphics
Tracking
()
RESOLVED
FIXED
mozilla14
People
(Reporter: tete009+bugzilla, Assigned: joe)
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
patch
|
jrmuizel
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0) Gecko/20100101 Firefox/11.0
Build ID: 20120312181643
Steps to reproduce:
In Bug 685767, we moved innards of gfxAlphaBoxBlur to mozilla::gfx::AlphaBoxBlur.
Actual results:
Fixedpoint division introduced by Bug 633627 was replaced to integer division by Bug 685767.
Expected results:
We should apply the patch of fixedpoint division of Bug 633627 again.
Reporter | ||
Updated•13 years ago
|
Component: Untriaged → Graphics
Keywords: regression
OS: Windows 7 → All
Product: Firefox → Core
Hardware: x86_64 → All
Reporter | ||
Updated•13 years ago
|
Updated•13 years ago
|
Assignee: nobody → joe
Assignee | ||
Updated•13 years ago
|
QA Contact: untriaged → thebes
Assignee | ||
Comment 1•13 years ago
|
||
This is just taking the patch from bug 633627 and applying it to gfx/2d/Blur.cpp.
Attachment #610225 -
Flags: review?(jmuizelaar)
Assignee | ||
Updated•13 years ago
|
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Updated•13 years ago
|
Attachment #610225 -
Flags: review?(jmuizelaar) → review+
Reporter | ||
Updated•13 years ago
|
Keywords: checkin-needed
Comment 2•13 years ago
|
||
Comment 3•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•