42944 2010-07-25 03:35:43 -0700 SVG background doesn't resize properly when dimensions are changed 2011-10-31 08:32:27 -0700 1 1 1 Unclassified WebKit SVG 528+ (Nightly build) Mac (Intel) OS X 10.6 RESOLVED FIXED P2 Normal --- 47156 1 radexpl webkit-unassigned adele krit mitz radexpl thorton zimmermann oldest_to_newest 255810 0 62522 radexpl 2010-07-25 03:35:43 -0700 Created attachment 62522 Bug test Overview: Given a div with SVG background and border, div's background doesn't resize when div's dimensions are changed (no matter if using JS animation, CSS transition, or just :hover) Steps to reproduce: 1. Make a <div> with SVG background 2. Add a border or an outline to this <div> (bug won't appear without this) 3. Change its dimensions in some way (easiest way is to use :hover) Actual results: Border changes its dimensions, but background stays in the same position Expected results: Background changes its dimensions accordingly to div Build Date & Platform: Safari 5.0 (6533.16) - which is WebKit 533, right? Additional Builds and Platforms: Works just fine on Opera 10.60 Additional Information: Sorry for my English ;) 256409 1 krit 2010-07-26 23:48:03 -0700 I can't reproduce it locally with trunk. Have you tested it with a nightly? 256416 2 radexpl 2010-07-27 00:08:02 -0700 (In reply to comment #1) > I can't reproduce it locally with trunk. Have you tested it with a nightly? No until now (sorry for that - I'm the new one), but I can reproduce it on nightly. 258080 3 zimmermann 2010-07-30 00:13:51 -0700 CC'in Dan Bernstein. Dan, do you know off-hand what might be the difference that the SVG background image doesn't resize anymore when a border is applied to the <div>? I'd be thankful for any hints. 424580 4 thorton 2011-06-21 11:14:56 -0700 As far as I can tell, this is a regression in r62922, which added RenderSVGRoot ::clippedOverflowRectForRepaint, which redirects the call to SVGRenderSupport instead of the superclass, RenderBox. SVGRenderSupport's implementation uses the m_repaintBoundingBox on the RenderSVGRoot, which is calculated using computeContainerBoundingBoxes, which only takes into account the bounding boxes of the children of the container! (that is to say, ignores the bounding box of the RenderSVGRoot, which includes the background!) We could fix this by making RenderSVGRoot::updateCachedBoundaries include the RenderSVGRoot's bounding box *in the case where it has a background*, but I'm not sure if that would break anything. Alternatively, we could just remove RenderSVGRoot::clippedOverflowRectForRepaint, leaving it to be forwarded to RenderBox, but I have a feeling it's done the way it is right now for performance reasons, so we should be careful with it. Any ideas? <rdar://problem/9474533> 424892 5 thorton 2011-06-21 17:23:38 -0700 Nevermind! This is actually separate from the r62922 regression, which I'll file a separate bug for tomorrow because it needs some discussion. Sorry! 441367 6 zimmermann 2011-07-23 05:01:39 -0700 47156 fixes this. 493324 7 zimmermann 2011-10-31 08:32:27 -0700 Fixed in r98852. Thanks for the testcase! 62522 2010-07-25 03:35:43 -0700 2010-07-25 03:35:43 -0700 Bug test animated-svg-background-bug.zip application/zip 2441 radexpl UEsDBAoAAAAAAPJj+TwAAAAAAAAAAAAAAAAcABAAYW5pbWF0ZWQtc3ZnLWJhY2tncm91bmQtYnVn L1VYDACsB0xMhxJMTPUBFABQSwMEFAAIAAgA8mP5PAAAAAAAAAAAAAAAACUAEABhbmltYXRlZC1z dmctYmFja2dyb3VuZC1idWcvLkRTX1N0b3JlVVgMAO8STEyHEkxM9QEUAO2YOw7CMBBEZ40LSzQu Kd1wAG5gRckJuAAFV6D30SHaEbIUUlAlgnmS9VaKf2kcTwDY8LhfgAwgwY0zPpLYFoSuNs4hhBBC iH1jrnTcdhtCiB0ynw+FrnRzG58HOnZjMl3oSje3sV+gI53oTBe60s3NQ8sYPowrGxOKMYVYoetX ryzE33Bw5fn7P2E1/wshfhiL43Uc8A4Eyw6vduvqhvVLQPCfhadubKEr3dy6CAixFU9QSwcIagCI bbIAAAAEGAAAUEsDBAoAAAAAACpk+TwAAAAAAAAAAAAAAAAJABAAX19NQUNPU1gvVVgMAO8STEzv EkxM9QEUAFBLAwQKAAAAAAAqZPk8AAAAAAAAAAAAAAAAJQAQAF9fTUFDT1NYL2FuaW1hdGVkLXN2 Zy1iYWNrZ3JvdW5kLWJ1Zy9VWAwA7xJMTO8STEz1ARQAUEsDBBQACAAIAPJj+TwAAAAAAAAAAAAA AAAwABAAX19NQUNPU1gvYW5pbWF0ZWQtc3ZnLWJhY2tncm91bmQtYnVnLy5fLkRTX1N0b3JlVVgM AO8STEyHEkxM9QEUAGNgFWNnYGJg8E1MVvAPVohQgAKQGAMnEBsBsQIQg/hBDEjAgQE7AABQSwcI 3JGPuCgAAABSAAAAUEsDBBQACAAIAJpe+TwAAAAAAAAAAAAAAAAlABAAYW5pbWF0ZWQtc3ZnLWJh Y2tncm91bmQtYnVnL2ltYWdlLnN2Z1VYDADvEkxMcwlMTPUBFACzKS5LVyjPTCnJsFUyNDBQVVLI SM1MzyiB8cpSi4oz8/OAXD1DJa6K3Jy8YluljJKSAit9/fLycr1yY738onR9IwMDA32gUUp2XFw2 RanJJfjMLC6pzEm1VUrLzMmxKkpP0jA0stCBYk0lfZAJIKPsAFBLBwiZKbkrcQAAAJsAAABQSwME FAAIAAgAml75PAAAAAAAAAAAAAAAADAAEABfX01BQ09TWC9hbmltYXRlZC1zdmctYmFja2dyb3Vu ZC1idWcvLl9pbWFnZS5zdmdVWAwA7xJMTHMJTEz1ARQAY2AVY2dgYmDwTUxW8A9WiFCAApAYAycQ GwFxBxCD+LsYiAKOISFBDEnZrFAdM4BYCU0JI0JcNDk/Vy83MbkoPzexJLVYLzmxKLWEoZoL5Irk /JzS3DwFWwUzA2uwQE5mXiqQa2zNVQsAUEsHCOhXBqlpAAAAugAAAFBLAwQUAAgACADhYvk8AAAA AAAAAAAAAAAAJgAQAGFuaW1hdGVkLXN2Zy1iYWNrZ3JvdW5kLWJ1Zy9pbmRleC5odG1sVVgMAO8S TEyGEExM9QEUAFWP0QrCMAxF3/sVURF1oJuCL13dv3RrbIvVlrabG7J/dyvzwZBAbuBeTthK2CYO DkHFp6kIC3EwWJFNzT35EABQqKWK9HItXF/Oh7cWUdFzsU1qntp6gZ6eXQ/BGi2gNrx5lPBXeQZR 6QBTv7DBELgf4G491K2EaIE7h9xDlqfEyS+9bV/iqJ9cIm292e/Segqd3B1KMpIESZXtcEH9kS1o C/m1SOQjYfnyHBO6Ay1u68m/rlg+yeoLUEsHCHrkFYi5AAAACQEAAFBLAwQUAAgACADhYvk8AAAA AAAAAAAAAAAAMQAQAF9fTUFDT1NYL2FuaW1hdGVkLXN2Zy1iYWNrZ3JvdW5kLWJ1Zy8uX2luZGV4 Lmh0bWxVWAwA7xJMTIYQTEz1ARQAY2AVY2dgYmDwTUxW8A9WiFCAApAYAycQGwFxOxCD+DsZiAKO ISFBDEnZjFAdM4BYEU0JI0JcNDk/Vy83MbkoPzexJLVYLzmxKLWEoZoL5Irk/JzS3DwFWwVLazA/ JzMvFcgzt+aqBQBQSwcIu/NrRGgAAAC5AAAAUEsBAhUDCgAAAAAA8mP5PAAAAAAAAAAAAAAAABwA DAAAAAAAAAAAQO1BAAAAAGFuaW1hdGVkLXN2Zy1iYWNrZ3JvdW5kLWJ1Zy9VWAgArAdMTIcSTExQ SwECFQMUAAgACADyY/k8agCIbbIAAAAEGAAAJQAMAAAAAAAAAABApIFKAAAAYW5pbWF0ZWQtc3Zn LWJhY2tncm91bmQtYnVnLy5EU19TdG9yZVVYCADvEkxMhxJMTFBLAQIVAwoAAAAAACpk+TwAAAAA AAAAAAAAAAAJAAwAAAAAAAAAAED9QV8BAABfX01BQ09TWC9VWAgA7xJMTO8STExQSwECFQMKAAAA AAAqZPk8AAAAAAAAAAAAAAAAJQAMAAAAAAAAAABA/UGWAQAAX19NQUNPU1gvYW5pbWF0ZWQtc3Zn LWJhY2tncm91bmQtYnVnL1VYCADvEkxM7xJMTFBLAQIVAxQACAAIAPJj+TzckY+4KAAAAFIAAAAw AAwAAAAAAAAAAECkgekBAABfX01BQ09TWC9hbmltYXRlZC1zdmctYmFja2dyb3VuZC1idWcvLl8u RFNfU3RvcmVVWAgA7xJMTIcSTExQSwECFQMUAAgACACaXvk8mSm5K3EAAACbAAAAJQAMAAAAAAAA AABApIF/AgAAYW5pbWF0ZWQtc3ZnLWJhY2tncm91bmQtYnVnL2ltYWdlLnN2Z1VYCADvEkxMcwlM TFBLAQIVAxQACAAIAJpe+TzoVwapaQAAALoAAAAwAAwAAAAAAAAAAECkgVMDAABfX01BQ09TWC9h bmltYXRlZC1zdmctYmFja2dyb3VuZC1idWcvLl9pbWFnZS5zdmdVWAgA7xJMTHMJTExQSwECFQMU AAgACADhYvk8euQViLkAAAAJAQAAJgAMAAAAAAAAAABApIEqBAAAYW5pbWF0ZWQtc3ZnLWJhY2tn cm91bmQtYnVnL2luZGV4Lmh0bWxVWAgA7xJMTIYQTExQSwECFQMUAAgACADhYvk8u/NrRGgAAAC5 AAAAMQAMAAAAAAAAAABApIFHBQAAX19NQUNPU1gvYW5pbWF0ZWQtc3ZnLWJhY2tncm91bmQtYnVn Ly5faW5kZXguaHRtbFVYCADvEkxMhhBMTFBLBQYAAAAACQAJAFUDAAAeBgAAAAA=