Bug 141997 – Cleanup BackgroundImageGeometry class.

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

[zalan]

and RenderBoxModelObject::calculateBackgroundImageGeometry.

[zalan]

Created attachment 247297 [details]
Patch

[Build Bot]

Comment on attachment 247297 [details]
Patch

Attachment 247297 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/6070849257865216

New failing tests:
fast/backgrounds/hidpi-bitmap-background-repeat-on-subpixel-position.html
fast/shapes/shape-outside-floats/shape-outside-floats-image-margin-003.html
imported/mozilla/svg/as-image/background-stretch-1.html
ietestcenter/css3/flexbox/flexbox-pack-center-001.htm
css3/masking/mask-repeat-space-border.html
css3/background/background-repeat-space-padding.html
css3/masking/mask-repeat-round-auto2.html
fast/shapes/shape-outside-floats/shape-outside-floats-image-threshold-001.html
css3/background/background-repeat-round-auto2.html
fast/shapes/shape-outside-floats/shape-outside-floats-image-002.html
css3/background/background-repeat-space-border.html
ietestcenter/css3/flexbox/flexbox-align-baseline-001.htm
ietestcenter/css3/flexbox/flexbox-flex-001.htm
css3/masking/mask-svg-clipped-fragmentId.html
fast/shapes/shape-outside-floats/shape-outside-floats-image-001.html
css3/masking/mask-repeat-space-padding.html
fast/css/cascade/background-origin-and-webkit-background-origin-cascade-order.html
fast/backgrounds/hidpi-bitmap-background-origin-on-subpixel-position.html
fast/backgrounds/background-position-rendering.html
fast/shapes/shape-outside-floats/shape-outside-image-set.html
css3/masking/mask-repeat-space-content.html
css3/masking/mask-base64.html

[Build Bot]

Created attachment 247305 [details]
Archive of layout-test-results from ews100 for mac-mavericks

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews100  Port: mac-mavericks  Platform: Mac OS X 10.9.5

[Build Bot]

Comment on attachment 247297 [details]
Patch

Attachment 247297 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5389136043180032

New failing tests:
css3/masking/mask-repeat-space-padding.html
css3/masking/mask-svg-clipped-fragmentId.html
fast/backgrounds/background-position-rendering.html
fast/backgrounds/hidpi-bitmap-background-repeat-on-subpixel-position.html
css3/background/background-repeat-space-padding.html
css3/masking/mask-repeat-round-auto2.html
fast/shapes/shape-outside-floats/shape-outside-floats-image-threshold-001.html
ietestcenter/css3/flexbox/flexbox-flex-001.htm
imported/mozilla/svg/as-image/background-stretch-1.html
css3/masking/mask-base64.html
css3/background/background-repeat-round-auto2.html
fast/shapes/shape-outside-floats/shape-outside-floats-image-002.html
ietestcenter/css3/flexbox/flexbox-pack-center-001.htm
css3/masking/mask-repeat-space-content.html
fast/shapes/shape-outside-floats/shape-outside-floats-image-001.html
css3/background/background-repeat-space-border.html
ietestcenter/css3/flexbox/flexbox-align-baseline-001.htm
fast/css/cascade/background-origin-and-webkit-background-origin-cascade-order.html
fast/backgrounds/hidpi-bitmap-background-origin-on-subpixel-position.html
fast/shapes/shape-outside-floats/shape-outside-image-set.html
fast/shapes/shape-outside-floats/shape-outside-floats-image-margin-003.html

[Build Bot]

Created attachment 247306 [details]
Archive of layout-test-results from ews106 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5

[zalan]

Created attachment 247342 [details]
Patch

[Simon Fraser (smfr)]

Comment on attachment 247342 [details]
Patch

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

> Source/WebCore/rendering/RenderBoxModelObject.cpp:1164
> +        phase.setWidth(tileSize.width() ? tileSize.width() -  fmodf(computedXPosition + left, tileSize.width()): 0);

two spaces

[WebKit Commit Bot]

Comment on attachment 247342 [details]
Patch

Clearing flags on attachment: 247342

Committed r180644: <http://trac.webkit.org/changeset/180644>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.