1094480 - CSS borders are broken if only some of the corners have a border-radius and the border-radius is greater than the sum of the border-radius and the width/height of the element's content area

Status: VERIFIED FIXED

(Core :: Layout: Block and Inline, defect)

Description

[Christopher Van Wiemeersch [:cvan]]

For some reason, mixing `border-radius` with a `border` seems to cause an odd layout effect now.

––

Test case:
http://jsbin.com/kevehi/8

Or load this in your browser:
data:text/html,<style>body { border: 15px solid darkcyan; border-radius: 0 30px 30px 0; height: 100px; width: 0; }</style>

––

Reproducible in Firefox 36.0a1 (2014-11-05) on Mac OS X 10.9.5

Not reproducible in Firefox 34.0 and lower on Mac OS X 10.9.5
Not reproducible in Firefox 33.0 and lower on Ubuntu 12.04
Not reproducible in Firefox 33.0 and lower on Windows 7

Not reproducible in Safari Version 7.1 (9537.85.10.17.1) nor Chrome Version 38.0.2125.111 on Mac OS X 10.9.5

––

Comment 1

[Christopher Van Wiemeersch [:cvan]]

Attachment: screenshot of border-radius/border bug. left: Firefox 36.0a1 (2014-11-05) on Mac OS X 10.9.5; right: Firefox 34.0 on Mac OS X 10.9.5

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Is this Mac-only?

Comment 3

[Christopher Van Wiemeersch [:cvan]]

(In reply to David Baron [:dbaron] (UTC-8) (needinfo? for questions) from comment #2)
> Is this Mac-only?

It was first discovered in Firefox 36 on Windows 7 (in bug 1094232 comment 0).

ATM, I don't have access to a Windows device with Firefox 36.

Comment 4

[Boris Zbarsky [:bzbarsky]]

[Tracking Requested - why for this release]:

This is a recent regression.  I see it in a tip-ish build, but not a 2014-10-30 nightly.

Comment 5

[Boris Zbarsky [:bzbarsky]]

The patch for bug 1094063 changes behavior here, but the result is still wrong...

Comment 6

[Boris Zbarsky [:bzbarsky]]

Regression range on nightlies: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=da125623d9cb&tochange=5dde8ea48fef

I'm guessing this is also a regression from bug 1091787.

Comment 7

[Jonathan Watt [:jwatt]]

Attachment: patch

Put the fill rule back to it's previous value.

Comment 8

[Jonathan Watt [:jwatt]]

Comment on attachment 8518356 [details] [diff] [review]
patch

https://hg.mozilla.org/integration/mozilla-inbound/rev/9192c3c0d10d

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/9192c3c0d10d

Comment 10

[Jonathan Watt [:jwatt]]

cvan: Thanks for filing with a reduced testcase.

Comment 11

[Teodora Vermesan (:TeoVermesan)]

Reproduced the issue on Firefox 36 Nightly (build id: 20141105030203) on Ubuntu 12.04, Mac OS X 10.8 and Windows 7 x64, verified as fixed on Firefox 36 Beta 7 (build id: 20150205114429) under Ubuntu 12.04 64 bit, Mac OS X 10.8 and Windows 7 x64