Bug 86062

Summary: [Gtk] Two SVG reftests failing after r116498
Product: WebKit Reporter: Zan Dobersek <zan>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED INVALID    
Severity: Normal CC: ap, darin, thorton, zimmermann
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
svg/text/add-tspan-position-bug.html image diff
none
svg/text/modify-tspan-position-bug.html image diff none

Zan Dobersek
Reported 2012-05-10 00:06:28 PDT
The following SVG reftests are consistently failing with image mismatches: svg/text/add-tspan-position-bug.html svg/text/modify-tspan-position-bug.html They were both added in r116498.
Attachments
svg/text/add-tspan-position-bug.html image diff (4.71 KB, image/png)
2012-05-10 00:26 PDT, Zan Dobersek 		no flags
Details
svg/text/modify-tspan-position-bug.html image diff (4.71 KB, image/png)
2012-05-10 00:27 PDT, Zan Dobersek 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Zan Dobersek
Comment 1 2012-05-10 00:26:35 PDT
Created attachment 141096 [details] svg/text/add-tspan-position-bug.html image diff
Zan Dobersek
Comment 2 2012-05-10 00:27:14 PDT
Created attachment 141098 [details] svg/text/modify-tspan-position-bug.html image diff
Alexey Proskuryakov
Comment 3 2012-05-10 11:53:36 PDT
Please feel free to always CC author and reviewer of an offending change.
Tim Horton
Comment 4 2012-05-10 12:10:09 PDT
Can you post screenshots of what loading the two tests manually looks like, please?
Zan Dobersek
Comment 5 2012-05-11 00:09:13 PDT
(In reply to comment #4) > Can you post screenshots of what loading the two tests manually looks like, please? When loading manually the text is displayed and laid out as expected in both tests. Delaying the layoutTestController.notifyDone() call a bit (tested 50 and 100 ms) for a redraw to happen seems to help, the tests pass then.
Zan Dobersek
Comment 6 2012-05-12 00:25:21 PDT
The tests were rolled out in r116801, marking this bug invalid. Should be reopened when/if the changes are again committed into the tree. 1: http://trac.webkit.org/changeset/116801
Zan Dobersek
Comment 7 2012-05-12 08:44:28 PDT
Just as an additional note, the problem is fixable by doing another repaint before dumping pixel results in DRT. This is obviously not a problem introduced in r116498 but will rather be handled in bug #86284.
Note You need to log in before you can comment on or make changes to this bug.