Bug 67298 - [Qt] getBBox() on a SVGPathElement with curves incorrectly includes control points
Summary: [Qt] getBBox() on a SVGPathElement with curves incorrectly includes control p...
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks: 67280
  Show dependency treegraph
 
Reported: 2011-08-31 10:29 PDT by Balazs Kelemen
Modified: 2014-02-03 03:18 PST (History)
3 users (show)

See Also:


Attachments
Minimal test case (303 bytes, image/svg+xml)
2013-07-15 18:06 PDT, Neil
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Balazs Kelemen 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?
Comment 1 Andreas Kling 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..
Comment 2 Balazs Kelemen 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.
Comment 3 Tim Horton 2011-11-30 12:09:46 PST
By the way, this landed again in http://trac.webkit.org/changeset/99460
Comment 4 Rob Buis 2012-04-30 15:19:05 PDT
This problem maybe causes bug 71766.
Comment 5 Neil 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.)
Comment 6 Jocelyn Turcotte 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.