Summary: | [Qt] Modifying SVG path dumping to equal to other ports | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Csaba Osztrogonác <ossy> | ||||
Component: | SVG | Assignee: | QtWebKit Unassigned <webkit-qt-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | abecsi, kent.hansen, kling, krit, tonikitoo, webkit.review.bot, zimmermann | ||||
Priority: | P3 | Keywords: | Qt | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Csaba Osztrogonác
2010-01-17 23:41:34 PST
Hi Csaba, thanks for the bug report. We should really rewrite to have a cross-platform Path::debugString() implementation, by supplying a path-applier function, that's used to iterate over all path segments (using Path::apply()) in order to dump them. Shouldn't be much work, does anyone volunteer? :-) (In reply to comment #1) > Hi Csaba, > > thanks for the bug report. We should really rewrite to have a cross-platform > Path::debugString() implementation, by supplying a path-applier function, > that's used to iterate over all path segments (using Path::apply()) in order to > dump them. > > Shouldn't be much work, does anyone volunteer? :-) Also it is not possible for qt to match the LayoutTest results. Qt closes paths, once they cross each other, while SVG doesn't. Not a problem for the visible result, but gives wrong output for DRT. Qt also closes path's automaticly, if start and ending point match each other. This causes one of the W3C tests to fail. Altogether, it's realy better to follow Nikos advice, even if I still did not understand how he wants to do it ;-) Created attachment 48236 [details]
make path dump closer to Mac
Now I can't produce a cross-platform Path::debugString() implementation,
but I would like to make our expected files more similar to Mac.
Proposed patch to modify path dumping and update expected files attached.
Attachment 48236 [details] did not pass style-queue:
Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebCore/platform/graphics/qt/PathQt.cpp:314: Missing space after , [whitespace/comma] [3]
WebCore/platform/graphics/qt/PathQt.cpp:317: Missing space after , [whitespace/comma] [3]
WebCore/platform/graphics/qt/PathQt.cpp:327: Missing space after , [whitespace/comma] [3]
Total errors found: 3
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 48236 [details]
make path dump closer to Mac
r=me, but please add spaces after the commas in arg(...)
Comment on attachment 48236 [details] make path dump closer to Mac Style fixed, long line splitted as Kenneth asked on #qtwebkit, and then patch landed in http://trac.webkit.org/changeset/54429 Can we close the bug now? (In reply to comment #7) > Can we close the bug now? I think we shouldn't close this bug. Not yet. Because we have some more little differences between Mac and Qt SVG path dumping results. Additionally we have a strange problem because 0.00 isn't equal to -0.00. It needs more investigation. This was fixed by <http://trac.webkit.org/changeset/69386> since we no longer dump platform-specific data for RenderSVGPath. |