[SVG2] overflow should be ignored in img and background-image.

[SVG2] overflow should be ignored in img and background-image.

BUG=371250
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=173752

[fs, 2014-05-08 14:02:18]

On 2014/05/08 12:32:07, Erik Dahlström wrote:

Testcase?

[Erik Dahlström (inactive), 2014-05-08 15:32:57]

On 2014/05/08 14:02:18, fs wrote:
> On 2014/05/08 12:32:07, Erik Dahlström wrote:
> 
> Testcase?

Added. PTAL.

[fs, 2014-05-09 12:18:27]

LGTM w/ nit.

https://codereview.chromium.org/277563004/diff/40001/LayoutTests/svg/as-backg...
File LayoutTests/svg/as-background-image/overflow-auto-expected.html (right):

https://codereview.chromium.org/277563004/diff/40001/LayoutTests/svg/as-backg...
LayoutTests/svg/as-background-image/overflow-auto-expected.html:7:
background-image:
url(data:image/svg+xml;base64,PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0idXRmLTgiPz4KPCEtLSBHZW5lcmF0b3I6IEFkb2JlIElsbHVzdHJhdG9yIDE3LjEuMCwgU1ZHIEV4cG9ydCBQbHVnLUluIC4gU1ZHIFZlcnNpb246IDYuMDAgQnVpbGQgMCkgIC0tPgo8IURPQ1RZUEUgc3ZnIFBVQkxJQyAiLS8vVzNDLy9EVEQgU1ZHIDEuMS8vRU4iICJodHRwOi8vd3d3LnczLm9yZy9HcmFwaGljcy9TVkcvMS4xL0RURC9zdmcxMS5kdGQiPgo8c3ZnIHZlcnNpb249IjEuMSIgaWQ9IkxheWVyXzEiIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3N2ZyIgeG1sbnM6eGxpbms9Imh0dHA6Ly93d3cudzMub3JnLzE5OTkveGxpbmsiIHg9IjBweCIgeT0iMHB4IgoJIHdpZHRoPSI0MHB4IiBoZWlnaHQ9IjQwcHgiIHZpZXdCb3g9IjAgMCA0MCA0MCIgZW5hYmxlLWJhY2tncm91bmQ9Im5ldyAwIDAgNDAgNDAiIHhtbDpzcGFjZT0icHJlc2VydmUiIG92ZXJmbG93PSJoaWRkZW4iPgo8bGluZWFyR3JhZGllbnQgaWQ9IlNWR0lEXzFfIiBncmFkaWVudFVuaXRzPSJ1c2VyU3BhY2VPblVzZSIgeDE9IjUuODU3OSIgeTE9IjUuODU3OSIgeDI9IjM0LjE0MjEiIHkyPSIzNC4xNDIxIiBncmFkaWVudFRyYW5zZm9ybT0ibWF0cml4KDAuNzA3MSAwLjcwNzEgLTAuNzA3MSAwLjcwNzEgMjAgLTguMjg0MykiPgoJPHN0b3AgIG9mZnNldD0iMCIgc3R5bGU9InN0b3AtY29sb3I6IzAwQTk5NSIvPgoJPHN0b3AgIG9mZnNldD0iMCIgc3R5bGU9InN0b3AtY29sb3I6IzNDMzc5NSIvPgoJPHN0b3AgIG9mZnNldD0iMC41MDU2IiBzdHlsZT0ic3RvcC1jb2xvcjojQTUzNTk0Ii8+Cgk8c3RvcCAgb2Zmc2V0PSIxIiBzdHlsZT0ic3RvcC1jb2xvcjojREEyMTI4Ii8+CjwvbGluZWFyR3JhZGllbnQ+Cjxwb2x5Z29uIGZpbGw9InVybCgjU1ZHSURfMV8pIiBwb2ludHM9IjIwLDQwIDAsMjAgMjAsMCA0MCwyMCAiLz4KPC9zdmc+Cg==);
Nit: It's non-obvious how this data-URI differs from the one in the
non-reference, so some sort of comment wouldn't hurt I think. (I assume one is
'overflow: auto' and the other some other value on overflow?)

[Erik Dahlström (inactive), 2014-05-09 12:41:00]

The CQ bit was checked by ed@opera.com

[commit-bot: I haz the power, 2014-05-09 12:42:38]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ed@opera.com/277563004/60001

[commit-bot: I haz the power, 2014-05-09 14:02:28]

Change committed as 173752