Closed
Bug 1180234
Opened 9 years ago
Closed 9 years ago
Reftest for bug 1174332 fails with some GTK3 themes
Categories
(Core :: Widget: Gtk, defect)
Core
Widget: Gtk
Tracking
()
RESOLVED
FIXED
mozilla42
Tracking | Status | |
---|---|---|
firefox42 | --- | fixed |
People
(Reporter: acomminos, Assigned: acomminos)
References
Details
Attachments
(2 files)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
mstange
:
review+
|
Details | Diff | Splinter Review |
The way the reftest for bug 1174332 calculates the bounds for the box shadow causes some issues on GTK3 themes that draw partially transparent focused checkboxes, such as the version of Ambiance on the test infrastructure. The flood bounds for the checkbox in the reference are calculated as follows; > <feMorphology operator="dilate" radius="5" in="SourceAlpha" result="mask"/> However, this produces unexpected results with Ambiance's focused checkbox. The attachment makes it clear why that might be so.
Assignee | ||
Comment 1•9 years ago
|
||
The issue here is that the box-shadow test draws the checkbox directly on the blue background, causing accumulated error with multiple compositions by the theme engine (it composites each part of the checkbox onto the background, such as the background, outline, focus ring, etc). The SVG reference instead draws the widget on a temporary clear surface, and composites that onto the background. This issue is present throughout many reftests that we fuzz when gtkWidget is true. This patch adds extra fuzziness for the tryserver's GTK3 theme.
Attachment #8631249 -
Flags: review?(mstange)
Updated•9 years ago
|
Attachment #8631249 -
Flags: review?(mstange) → review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Comment 3•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/cb1d12d4ccfd
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox42:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
Assignee | ||
Updated•9 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•