[webkit-reviews] review granted: [Bug 135581] SSO expects to be able to walk parent application's bundle : [Attachment 235992] Patch

[bugzilla-daemon at webkit.org]

Alexey Proskuryakov <ap at webkit.org> has granted Oliver Hunt
<oliver at apple.com>'s request for review:
Bug 135581: SSO expects to be able to walk parent application's bundle
https://bugs.webkit.org/show_bug.cgi?id=135581

Attachment 235992: Patch
https://bugs.webkit.org/attachment.cgi?id=235992&action=review

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=235992&action=review


Weird, but OK.

>
Source/WebKit2/Resources/SandboxProfiles/ios/com.apple.WebKit.Networking.sb:32
> -;; FIXME: Remove the webkti specific extension classes
<rdar://problem/17755931>
> +;; FIXME: Remove the webkit specific extension classes
<rdar://problem/17755931>

Do we actually have any of those?

> Source/WebKit2/UIProcess/mac/WebContextMac.mm:326
> +String WebContext::platformDefaultParentBundleDirectory() const

I'm still very annoyed by this seemingly cross platform code being added only
for iOS. These functions should be completely inside PLATFORM(IOS) checks,
having them defined with a broken fallback behavior is misleading and
unhelpful.