Bug 62007

Summary: Incorrect embedded document replaced size calculation for box-sizing: border-box
Product: WebKit Reporter: Nikolas Zimmermann <zimmermann>
Component: SVGAssignee: Nikolas Zimmermann <zimmermann>
Status: RESOLVED FIXED    
Severity: Normal CC: eric, kling, levin, zimmermann
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 34521    
Attachments:
Description Flags
Patch
none
Patch v2 kling: review+

Nikolas Zimmermann
Reported 2011-06-03 05:24:18 PDT
Incorrect embedded document replaced size calculation for box-sizing: border-box. Easy fix coming.
Attachments
Patch (86.33 KB, patch)
2011-06-03 05:35 PDT, Nikolas Zimmermann 		no flags
DetailsFormatted DiffDiff
Patch v2 (88.86 KB, patch)
2011-06-03 05:38 PDT, Nikolas Zimmermann 		kling: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Nikolas Zimmermann
Comment 1 2011-06-03 05:35:14 PDT
Created attachment 95896 [details] Patch
Nikolas Zimmermann
Comment 2 2011-06-03 05:38:30 PDT
Created attachment 95898 [details] Patch v2 Hum, no ChangeLog and the style queue is green?
Nikolas Zimmermann
Comment 3 2011-06-03 05:38:53 PDT
CC'ing Eric - is this a style queue bug? Shouldn't it moan about missing ChangeLogs?
Andreas Kling
Comment 4 2011-06-03 05:41:40 PDT
Comment on attachment 95898 [details] Patch v2 View in context: https://bugs.webkit.org/attachment.cgi?id=95898&action=review LGTM. > LayoutTests/svg/as-object/object-box-sizing-no-width-height.html:42 > + <!-- Part of this object is clipped, because of the diferent box sizing strategy, matches Opera/FF --> Typo, different.
Nikolas Zimmermann
Comment 5 2011-06-03 05:43:37 PDT
Thanks, landed in r88009.
Eric Seidel (no email)
Comment 6 2011-06-03 07:18:19 PDT
It should. Not sure why it didn't.
David Levin
Comment 7 2011-06-03 08:34:36 PDT
(In reply to comment #6) > It should. Not sure why it didn't. Here ya go: https://bugs.webkit.org/show_bug.cgi?id=62021 If anyone feels especially motivated to tackle it.... :)
Note You need to log in before you can comment on or make changes to this bug.