Bug 77428 – Tap highlighting: Support better outlines for multiline inlines

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[Kenneth Rohde Christiansen]

SSIA

[Kenneth Rohde Christiansen]

Created attachment 124714 [details]
Patch

[WebKit Review Bot]

Attachment 124714 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/update-webkit']" exit_code: 9

Updating OpenSource
From git://git.webkit.org/WebKit
   09fbdf8..3b3325c  master     -> origin/master
Partial-rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ...
Currently at 106354 = 09fbdf8bc1525cf6c284e230088f23103db232bd
r106355 = 3b3325cd5a20bf357b8fdad5c0c1dcb448125ec4
Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc
First, rewinding head to replay your work on top of it...
Applying: Fix compilation errors on build-webkit --debug --no-workers on mac.
Using index info to reconstruct a base tree...
Falling back to patching base and 3-way merge...
Auto-merging LayoutTests/ChangeLog
CONFLICT (content): Merge conflict in LayoutTests/ChangeLog
Auto-merging LayoutTests/platform/qt/Skipped
CONFLICT (content): Merge conflict in LayoutTests/platform/qt/Skipped
Auto-merging Source/WebCore/ChangeLog
CONFLICT (content): Merge conflict in Source/WebCore/ChangeLog
Failed to merge in the changes.
Patch failed at 0001 Fix compilation errors on build-webkit --debug --no-workers on mac.

When you have resolved this problem run "git rebase --continue".
If you would prefer to skip this patch, instead run "git rebase --skip".
To restore the original branch and stop rebasing run "git rebase --abort".

rebase refs/remotes/origin/master: command returned error: 1

Died at Tools/Scripts/update-webkit line 164.


If any of these errors are false positives, please file a bug against check-webkit-style.

[Simon Hausmann]

Comment on attachment 124714 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=124714&action=review

> Source/WebCore/page/GestureTapHighlighter.cpp:133
> +inline bool contains(LayoutRect rect, int x)

missing "static" keyword?

> Source/WebCore/page/GestureTapHighlighter.cpp:173
> +    if (!first.isEmpty()) {
> +        addRectWithRoundedCorners(path, first,
> +                true, true,
> +                !contains(middle, first.x()) && !contains(last, first.x()),
> +                !contains(middle, first.maxX()) && !contains(last, first.maxX()));

I wish there was a way to write this more readable, i.e. make it clearer what the meaning of the booleans here is.

[Kenneth Rohde Christiansen]

Corrected patch landed in 106357.