Should return an SkImage matching the container size, not the embedded SVG size.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8148eedf1595dab9ec126e3e20403bad53021db9 commit 8148eedf1595dab9ec126e3e20403bad53021db9 Author: fmalita <fmalita@chromium.org> Date: Thu Aug 04 18:59:45 2016 Fix SVGImage::imageForCurrentFrameForContainer() sizing SVGImageForContainer implements the Image interface, and reports a size equal to the container size. But its imageForCurrentFrame() implementation delegates to SVGImage::imageForCurrentFrameForContainer(), which currently returns an SkImage with a size equal to the SVG intrinsic size. This mismatch between the size reported by SVGImageForContainer and the size of the SkImage returned by its imageForCurrentFrame() is causing problems when tiling. The CL updates SVGImage::imageForCurrentFrameForContainer() to return an SkImage reflecting the container size instead of intrinsic size. New pixel test: fast/backgrounds/background-svg-scaling.html BUG= 632969 , 633637 R=schenney@chromium.org, pdr@chromium.org, fs@opera.com Review-Url: https://codereview.chromium.org/2203093003 Cr-Commit-Position: refs/heads/master@{#409853} [modify] https://crrev.com/8148eedf1595dab9ec126e3e20403bad53021db9/third_party/WebKit/LayoutTests/TestExpectations [add] https://crrev.com/8148eedf1595dab9ec126e3e20403bad53021db9/third_party/WebKit/LayoutTests/fast/backgrounds/background-svg-scaling.html [modify] https://crrev.com/8148eedf1595dab9ec126e3e20403bad53021db9/third_party/WebKit/LayoutTests/svg/canvas/canvas-default-object-sizing-expected.html [modify] https://crrev.com/8148eedf1595dab9ec126e3e20403bad53021db9/third_party/WebKit/Source/core/svg/graphics/SVGImage.cpp
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b104e8c464bbd8c2bb537ad4b6903990c8527357 commit b104e8c464bbd8c2bb537ad4b6903990c8527357 Author: Rebaseline Bot <blink-rebaseline-bot@chromium.org> Date: Thu Aug 04 20:30:04 2016 Auto-rebaseline for r409853 https://chromium.googlesource.com/chromium/src/+/8148eedf1 BUG= 633637 TBR=fmalita@chromium.org Review URL: https://codereview.chromium.org/2217473003 . Cr-Commit-Position: refs/heads/master@{#409888} [modify] https://crrev.com/b104e8c464bbd8c2bb537ad4b6903990c8527357/third_party/WebKit/LayoutTests/TestExpectations [add] https://crrev.com/b104e8c464bbd8c2bb537ad4b6903990c8527357/third_party/WebKit/LayoutTests/fast/backgrounds/background-svg-scaling-expected.png [add] https://crrev.com/b104e8c464bbd8c2bb537ad4b6903990c8527357/third_party/WebKit/LayoutTests/platform/linux/fast/backgrounds/background-svg-scaling-expected.png [add] https://crrev.com/b104e8c464bbd8c2bb537ad4b6903990c8527357/third_party/WebKit/LayoutTests/platform/linux/fast/backgrounds/background-svg-scaling-expected.txt [add] https://crrev.com/b104e8c464bbd8c2bb537ad4b6903990c8527357/third_party/WebKit/LayoutTests/platform/mac/fast/backgrounds/background-svg-scaling-expected.txt [add] https://crrev.com/b104e8c464bbd8c2bb537ad4b6903990c8527357/third_party/WebKit/LayoutTests/platform/win/fast/backgrounds/background-svg-scaling-expected.txt
Comment 1 by bugdroid1@chromium.org
, Aug 4 2016