WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
178611
Add tests to ensure spelling error dots are drawn in the correct place for overlapping lines
https://bugs.webkit.org/show_bug.cgi?id=178611
Summary
Add tests to ensure spelling error dots are drawn in the correct place for ov...
Daniel Bates
Reported
2017-10-20 17:03:30 PDT
Spelling error dots drawn in wrong place for overlapping lines.
Attachments
[Screenshot] Expected (shipping Safari on macOS Sierra)
(7.52 KB, image/png)
2017-10-20 17:05 PDT
,
Daniel Bates
no flags
Details
[Screenshot] Bug
(8.39 KB, image/png)
2017-10-20 17:06 PDT
,
Daniel Bates
no flags
Details
Example
(580 bytes, text/html)
2017-10-20 17:07 PDT
,
Daniel Bates
no flags
Details
Patch and layout tests
(10.36 KB, patch)
2017-10-20 17:33 PDT
,
Daniel Bates
zalan
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Daniel Bates
Comment 1
2017-10-20 17:05:29 PDT
Created
attachment 324471
[details]
[Screenshot] Expected (shipping Safari on macOS Sierra) Disregarding the orientation of the dots (this was fixed in
bug #176949
), this screenshot represents the expected position for the dots.
Daniel Bates
Comment 2
2017-10-20 17:06:09 PDT
Created
attachment 324472
[details]
[Screenshot] Bug Screenshot following
r223699
.
Daniel Bates
Comment 3
2017-10-20 17:07:21 PDT
Created
attachment 324473
[details]
Example
Radar WebKit Bug Importer
Comment 4
2017-10-20 17:08:37 PDT
<
rdar://problem/35105805
>
Daniel Bates
Comment 5
2017-10-20 17:33:03 PDT
Created
attachment 324477
[details]
Patch and layout tests
Daniel Bates
Comment 6
2017-10-23 10:41:48 PDT
As it turns outs I decided to revert
r223699
in <
https://trac.webkit.org/changeset/223840
> because it causes regressions in addition to the one reported in this bug. I will incorporate the source code changes from the patch on this bug (
attachment #324477
[details]
) into an updated patch for
bug #178232
. I am re-purposing this bug to land the layout tests included in
attachment #324477
[details]
. This will ensure that we do not regress painting of spelling dots in overlapping lines.
Daniel Bates
Comment 7
2017-10-23 10:49:30 PDT
Committed
r223842
: <
https://trac.webkit.org/changeset/223842
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug