Bug 43018 – Strange scrollbars with SVG image

• Status: RESOLVED
• Resolution: FIXED
• Component: SVG

[Pierre Rudloff]

The SVG image at this URL has strange disabled scrollbars around it :
http://tael67.free.fr/divers/svg_problem/
It seems to come from the width/height attribute of the <svg> tag. If I change them, there is no scrollbar anymore.

[Dirk Schulze]

I can just confirm it for Chromium on linux. Does it happen in Safari as well?

[Pierre Rudloff]

I can reproduce with Chrome 5.0.375.99, Safari 5.0 and the latest nightly of Webkit (all on Mac OS).

[Philip Rogers]

Created attachment 113530 [details]
Add test for strange scrollbars with non-integer scaled SVG image

I boiled this bug down to an issue with non-integer sized SVG images being scaled when used as an image. This was just recently fixed (with the LayoutUnits changes, I think) but no test-case covers the issue described in this bug.

[Philip Rogers]

Created attachment 115721 [details]
Add test for strange scrollbars with non-integer scaled SVG image

Cleaned up the patch and tracked down the change that fixed this: r98852.

Even if this patch isn't accepted, this bug should be marked as fixed.

[WebKit Review Bot]

Attachment 115721 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/update-webkit']" exit_code: 9

Updating OpenSource
RA layer request failed: OPTIONS of 'http://svn.webkit.org/repository/webkit': could not connect to server (http://svn.webkit.org) at /usr/lib/git-core/git-svn line 2295

Died at Tools/Scripts/update-webkit line 157.


If any of these errors are false positives, please file a bug against check-webkit-style.

[Nikolas Zimmermann]

Comment on attachment 115721 [details]
Add test for strange scrollbars with non-integer scaled SVG image

Thanks, looks good to me.

[WebKit Review Bot]

Comment on attachment 115721 [details]
Add test for strange scrollbars with non-integer scaled SVG image

Clearing flags on attachment: 115721

Committed r100778: <http://trac.webkit.org/changeset/100778>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.