[webkit-reviews] review granted: [Bug 226048] [iOS][FCR] <select> options are unnecessarily truncated : [Attachment 429234] Patch

[bugzilla-daemon at webkit.org]

Tim Horton <thorton at apple.com> has granted Aditya Keerthi
<akeerthi at apple.com>'s request for review:
Bug 226048: [iOS][FCR] <select> options are unnecessarily truncated
https://bugs.webkit.org/show_bug.cgi?id=226048

Attachment 429234: Patch

https://bugs.webkit.org/attachment.cgi?id=429234&action=review




--- Comment #3 from Tim Horton <thorton at apple.com> ---
Comment on attachment 429234
  --> https://bugs.webkit.org/attachment.cgi?id=429234
Patch

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

> Source/WebKit/UIProcess/ios/forms/WKFormSelectPicker.mm:477
> +#define UIMenuOptionsPrivateRemoveLineLimitForChildren (1 << 6)

I think you want a WK prefix here? Or maybe it's OK and the compiler won't
whine if it has the real one because it's a #define?

Also, does UIKit ignore the random flag bit if it /doesn't/ have this change?