Bug 25383 - Updating SVG size does not update CSS width/height
Summary: Updating SVG size does not update CSS width/height
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 25387
  Show dependency treegraph
 
Reported: 2009-04-24 16:08 PDT by Eric Seidel (no email)
Modified: 2009-04-24 16:54 PDT (History)
2 users (show)

See Also:


Attachments
Fix width/height updates from JS (15.91 KB, patch)
2009-04-24 16:08 PDT, Eric Seidel (no email)
no flags Details | Formatted Diff | Diff
Simpler patch, just the code change (5.86 KB, patch)
2009-04-24 16:17 PDT, Eric Seidel (no email)
simon.fraser: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2009-04-24 16:08:10 PDT
This can be seen by putting a border around an <svg> element and then changing the width/height via javascript.

I fixed this bug with an ugly hack (there is currently no better way) :(  And I've updated our tests to cover this case.
Comment 1 Eric Seidel (no email) 2009-04-24 16:08:27 PDT
Created attachment 29773 [details]
Fix width/height updates from JS

 28 files changed, 90 insertions(+), 29 deletions(-)
Comment 2 Eric Seidel (no email) 2009-04-24 16:17:13 PDT
Created attachment 29775 [details]
Simpler patch, just the code change

 7 files changed, 57 insertions(+), 4 deletions(-)
Comment 3 Eric Seidel (no email) 2009-04-24 16:54:57 PDT
Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	LayoutTests/ChangeLog
	M	LayoutTests/platform/mac/svg/custom/viewport-update2-expected.checksum
	M	LayoutTests/platform/mac/svg/custom/viewport-update2-expected.png
	M	LayoutTests/platform/mac/svg/custom/viewport-update2-expected.txt
	M	LayoutTests/svg/custom/viewport-update2.svg
	M	WebCore/ChangeLog
	M	WebCore/svg/SVGSVGElement.cpp
Committed r42854