Bug 62007 - Incorrect embedded document replaced size calculation for box-sizing: border-box
Summary: Incorrect embedded document replaced size calculation for box-sizing: border-box
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nikolas Zimmermann
URL:
Keywords:
Depends on:
Blocks: 34521
  Show dependency treegraph
 
Reported: 2011-06-03 05:24 PDT by Nikolas Zimmermann
Modified: 2011-06-03 08:34 PDT (History)
4 users (show)

See Also:


Attachments
Patch (86.33 KB, patch)
2011-06-03 05:35 PDT, Nikolas Zimmermann
no flags Details | Formatted Diff | Diff
Patch v2 (88.86 KB, patch)
2011-06-03 05:38 PDT, Nikolas Zimmermann
kling: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Nikolas Zimmermann 2011-06-03 05:24:18 PDT
Incorrect embedded document replaced size calculation for box-sizing: border-box.
Easy fix coming.
Comment 1 Nikolas Zimmermann 2011-06-03 05:35:14 PDT
Created attachment 95896 [details]
Patch
Comment 2 Nikolas Zimmermann 2011-06-03 05:38:30 PDT
Created attachment 95898 [details]
Patch v2

Hum, no ChangeLog and the style queue is green?
Comment 3 Nikolas Zimmermann 2011-06-03 05:38:53 PDT
CC'ing Eric - is this a style queue bug? Shouldn't it moan about missing ChangeLogs?
Comment 4 Andreas Kling 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.
Comment 5 Nikolas Zimmermann 2011-06-03 05:43:37 PDT
Thanks, landed in r88009.
Comment 6 Eric Seidel (no email) 2011-06-03 07:18:19 PDT
It should.  Not sure why it didn't.
Comment 7 David Levin 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.... :)