Bug 77428 - Tap highlighting: Support better outlines for multiline inlines
Summary: Tap highlighting: Support better outlines for multiline inlines
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Kenneth Rohde Christiansen
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-31 05:42 PST by Kenneth Rohde Christiansen
Modified: 2012-01-31 06:35 PST (History)
2 users (show)

See Also:


Attachments
Patch (8.02 KB, patch)
2012-01-31 05:49 PST, Kenneth Rohde Christiansen
hausmann: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kenneth Rohde Christiansen 2012-01-31 05:42:56 PST
SSIA
Comment 1 Kenneth Rohde Christiansen 2012-01-31 05:49:12 PST
Created attachment 124714 [details]
Patch
Comment 2 WebKit Review Bot 2012-01-31 05:51:03 PST
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.
Comment 3 Simon Hausmann 2012-01-31 06:07:33 PST
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.
Comment 4 Kenneth Rohde Christiansen 2012-01-31 06:35:14 PST
Corrected patch landed in 106357.