Bug 140219 – Move -webkit-tap-highlight-color / -webkit-overflow-scrolling / -webkit-touch-callout to the new StyleBuilder

• Status: RESOLVED
• Resolution: FIXED
• Component: CSS

[Chris Dumez]

Move -webkit-tap-highlight-color / -webkit-overflow-scrolling / -webkit-touch-callout to the new StyleBuilder

[Chris Dumez]

Created attachment 244227 [details]
Patch

[Darin Adler]

Comment on attachment 244227 [details]
Patch

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

> Source/WebCore/css/StyleBuilderConverter.h:1061
> +    return downcast<CSSPrimitiveValue>(value).getStringValue().lower() != "none";

Can we do this with equalIgnoringCase instead of lower? It's a shame to copy the string if it happens to have an uppercase character in it, just to do an equality comparison. More efficient still would be equalIgnoringASCIICase if we had an overload that takes the right type.

> Source/WebCore/css/StyleBuilderConverter.h:1063
> +#endif // PLATFORM(IOS)

In these short sequences, these comments on the #endif are ugly and don't seem to add clarity.

[WebKit Commit Bot]

Comment on attachment 244227 [details]
Patch

Rejecting attachment 244227 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-01', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 244227, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
    -> origin/master
Partial-rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ...
Currently at 178099 = 872690584ea2f35ace4928e38aaafb983e616a41
r178100 = 608d0716a1e22238a9f4bfd5213b38689a782b18
r178101 = 8acfb24eec38d82cd85c718f17d1bcd954336f64
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...
Fast-forwarded master to refs/remotes/origin/master.

Full output: http://webkit-queues.appspot.com/results/5872560549920768

[Chris Dumez]

Created attachment 244243 [details]
Patch

[Chris Dumez]

Created attachment 244244 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 244244 [details]
Patch

Clearing flags on attachment: 244244

Committed r178108: <http://trac.webkit.org/changeset/178108>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.