adding -moz-border-radius to an element causes the background to be drawn without dpi scaling

RESOLVED FIXED

Status

()

RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: db48x, Assigned: db48x)

Tracking

Trunk
x86
All
Points:
---
Bug Flags:
blocking1.9 ?
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Description

12 years ago
Created attachment 268478 [details]
if you see red then there's a bug

This is the trunk from 20070615. I have set layout.css.dpi to 144, which causes everything to be scaled by 200%. However, adding a border radius to an element causes it's background to be drawn as if the scaling was not in effect. This effects both it's position and it's size. I'll attach a testcase which can easily be made into a reftest.
Flags: blocking1.9?
(Assignee)

Comment 1

12 years ago
Created attachment 271347 [details] [diff] [review]
384576-1.diff
Assignee: nobody → db48x
Status: NEW → ASSIGNED
(Assignee)

Updated

12 years ago
Attachment #271347 - Flags: superreview?(roc)
Attachment #271347 - Flags: review?(sharparrow1)
(Assignee)

Comment 2

12 years ago
Created attachment 271353 [details] [diff] [review]
384576-2.diff

add a reftest. Note that the test is only effective if dpi scaling is in effect
Attachment #271347 - Attachment is obsolete: true
Attachment #271353 - Flags: superreview?(roc)
Attachment #271353 - Flags: review?(sharparrow1)
Attachment #271347 - Flags: superreview?(roc)
Attachment #271347 - Flags: review?(sharparrow1)

Comment 3

12 years ago
Comment on attachment 271353 [details] [diff] [review]
384576-2.diff

(As a suggestion for the future, please use diff -pu10; it makes things easier to review.)
Attachment #271353 - Flags: review?(sharparrow1) → review+
Attachment #271353 - Flags: superreview?(roc) → superreview+
(Assignee)

Comment 4

12 years ago
checked in
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Daniel: do please remember to flip the testsuite flag when a test gets committed, as happened here.  :-)
Flags: in-testsuite+
(Assignee)

Comment 6

12 years ago
Thanks Jeff, I'd forgotten about that.
You need to log in before you can comment on or make changes to this bug.