[webkit-reviews] review granted: [Bug 198693] [iOS] The default application name for user agent when requesting desktop content is still "Mobile/15E148" : [Attachment 371686] Patch

[bugzilla-daemon at webkit.org]

Tim Horton <thorton at apple.com> has granted Wenson Hsieh
<wenson_hsieh at apple.com>'s request for review:
Bug 198693: [iOS] The default application name for user agent when requesting
desktop content is still "Mobile/15E148"
https://bugs.webkit.org/show_bug.cgi?id=198693

Attachment 371686: Patch

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




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

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

> Source/WebKit/UIProcess/API/Cocoa/WKWebViewConfiguration.mm:506
> +    return _applicationNameForUserAgent.hasValue() ?
_applicationNameForUserAgent->get() : nil;

valueOr?