Bug 67298

Summary: [Qt] getBBox() on a SVGPathElement with curves incorrectly includes control points
Product: WebKit Reporter: Balazs Kelemen <kbalazs>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED INVALID    
Severity: Normal CC: kling, rwlbuis, thorton
Priority: P2 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 67280    
Attachments:
Description Flags
Minimal test case none

Balazs Kelemen
Reported 2011-08-31 10:29:29 PDT
Qt clone of #65939 and #53512. Andreas, I remember you hacked on bounding box calculation a few days ago, do you have an idea how to solve this bug on Qt?
Attachments
Minimal test case (303 bytes, image/svg+xml)
2013-07-15 18:06 PDT, Neil
no flags
Andreas Kling
Comment 1 2011-08-31 10:37:59 PDT
(In reply to comment #0) > Qt clone of #65939 and #53512. Andreas, I remember you hacked on bounding box calculation a few days ago, do you have an idea how to solve this bug on Qt? Yes, sir! I know just what to do about this..
Balazs Kelemen
Comment 2 2011-09-02 04:18:17 PDT
http://trac.webkit.org/changeset/94236 rolled out the original CG patch so maybe we should wait a bit with this.
Tim Horton
Comment 3 2011-11-30 12:09:46 PST
By the way, this landed again in http://trac.webkit.org/changeset/99460
Rob Buis
Comment 4 2012-04-30 15:19:05 PDT
This problem maybe causes bug 71766.
Neil
Comment 5 2013-07-15 18:06:05 PDT
Created attachment 206712 [details] Minimal test case Load the attached test case. In the console execute: document.getElementById('obj').getBBox() Expected result: SVGRect {height: 25, width: 50, y: 100, x: 100} Actual result: SVGRect {height: 31, width: 50, y: 97, x: 100} The red rectangle represents the reported size of the blue shape. (The red rectangle is static, it won't update when this bug is fixed.)
Jocelyn Turcotte
Comment 6 2014-02-03 03:18:50 PST
=== Bulk closing of Qt bugs === If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary. If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.
Note You need to log in before you can comment on or make changes to this bug.