Bug 134491 – Subpixel rendering: Pixel crack in breadcrumbs at devforums.apple.com.

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[zalan]

There’s a 1-pixel white vertical line going through the navigation breadcrumbs at the top of Developer Forums posts at devforums.apple.com. 
https://devforums.apple.com/community/safari/plugins


<rdar://problem/17499160>

[zalan]

Created attachment 234166 [details]
Patch

[Simon Fraser (smfr)]

Comment on attachment 234166 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=234166&action=review

> Source/WebCore/rendering/RenderBoxModelObject.cpp:1152
> -    LayoutUnit computedXPosition = minimumValueForLength(fillLayer->xPosition(), availableWidth, true);
> +    LayoutUnit computedXPosition = minimumValueForLength(fillLayer->xPosition(), availableWidth, false);

Classic boolean trap. No idea what true/false mean.

[zalan]

(In reply to comment #2)
> (From update of attachment 234166 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=234166&action=review
> 
> > Source/WebCore/rendering/RenderBoxModelObject.cpp:1152
> > -    LayoutUnit computedXPosition = minimumValueForLength(fillLayer->xPosition(), availableWidth, true);
> > +    LayoutUnit computedXPosition = minimumValueForLength(fillLayer->xPosition(), availableWidth, false);
> 
> Classic boolean trap. No idea what true/false mean.

bug 134495

[WebKit Commit Bot]

Comment on attachment 234166 [details]
Patch

Rejecting attachment 234166 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-02', 'validate-changelog', '--check-oops', '--non-interactive', 234166, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

ChangeLog entry in LayoutTests/ChangeLog contains OOPS!.

Full output: http://webkit-queues.appspot.com/results/5347005584900096

[zalan]

Committed r170646: <http://trac.webkit.org/changeset/170646>