<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - border-image-repeat:round output doesn't match other browsers"
href="https://bugs.webkit.org/show_bug.cgi?id=155955#c7">Comment # 7</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - border-image-repeat:round output doesn't match other browsers"
href="https://bugs.webkit.org/show_bug.cgi?id=155955">bug 155955</a>
from <span class="vcard"><a class="email" href="mailto:simon.fraser@apple.com" title="Simon Fraser (smfr) <simon.fraser@apple.com>"> <span class="fn">Simon Fraser (smfr)</span></a>
</span></b>
<pre>Please do. I started looking at this and it's a bit more involved than I thought.
Basically I think the fix should reflect this comment in Image::drawTiled():
// FIXME: These rules follow CSS border-image rules, but they should not be down here in Image.
so the 'space' and 'round' logic there needs to pushed up into NinePieceImage::computeTileScales(), maybe as part of scaleSlicesIfNeeded(). This probably requires sending 'spacing' and 'phase' down through GraphicsContext::drawTiledImage(). I think this code move is necessary to implement this part of the spec:
"The middle image's width is scaled by the same factor as the top image unless that factor is zero or infinity, in which case the scaling factor of the bottom is substituted, and failing that, the width is not scaled. The height of the middle image is scaled by the same factor as the left image unless that factor is zero or infinity, in which case the scaling factor of the right image is substituted, and failing that, the height is not scaled."</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>