Fix the internal iOS build after r229512
authorwenson_hsieh@apple.com <wenson_hsieh@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 12 Mar 2018 03:38:22 +0000 (03:38 +0000)
committerwenson_hsieh@apple.com <wenson_hsieh@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 12 Mar 2018 03:38:22 +0000 (03:38 +0000)
https://bugs.webkit.org/show_bug.cgi?id=183550

Reviewed by Zalan Bujtas.

Fixes a typo in a header import, as well as a nullability error. `SecTrustEvaluate()` takes in a nonnull outparam,
which was removed in r229512; this adds the outparam back, but does not consult its value when setting the
`infoDictionary`.

* UIProcess/ios/fullscreen/WKFullScreenWindowControllerIOS.mm:
(-[WKFullScreenWindowController _EVOrganizationName]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@229521 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/ChangeLog
Source/WebKit/UIProcess/ios/fullscreen/WKFullScreenWindowControllerIOS.mm

index 8a21a95..29cb10b 100644 (file)
@@ -1,3 +1,17 @@
+2018-03-11  Wenson Hsieh  <wenson_hsieh@apple.com>
+
+        Fix the internal iOS build after r229512
+        https://bugs.webkit.org/show_bug.cgi?id=183550
+
+        Reviewed by Zalan Bujtas.
+
+        Fixes a typo in a header import, as well as a nullability error. `SecTrustEvaluate()` takes in a nonnull outparam,
+        which was removed in r229512; this adds the outparam back, but does not consult its value when setting the
+        `infoDictionary`.
+
+        * UIProcess/ios/fullscreen/WKFullScreenWindowControllerIOS.mm:
+        (-[WKFullScreenWindowController _EVOrganizationName]):
+
 2018-03-10  Jer Noble  <jer.noble@apple.com>
 
         Improvements to fullscreen; new UI and security features
index d69d54c..239af2f 100644 (file)
@@ -29,8 +29,8 @@
 #import "WKFullScreenWindowControllerIOS.h"
 
 #import "UIKitSPI.h"
+#import "WKFullScreenViewController.h"
 #import "WKFullscreenStackView.h"
-#import "WKFullscreenViewController.h"
 #import "WKWebView.h"
 #import "WKWebViewInternal.h"
 #import "WKWebViewPrivate.h"
@@ -768,7 +768,8 @@ static const NSTimeInterval kAnimationDuration = 0.2;
     // If SecTrustCopyInfo returned NULL then it's likely that the SecTrustRef has not been evaluated
     // and the only way to get the information we need is to call SecTrustEvaluate ourselves.
     if (!infoDictionary) {
-        OSStatus err = SecTrustEvaluate(trust, NULL);
+        SecTrustResultType result = kSecTrustResultProceed;
+        OSStatus err = SecTrustEvaluate(trust, &result);
         if (err == noErr)
             infoDictionary = [(__bridge NSDictionary *)SecTrustCopyInfo(trust) autorelease];
         if (!infoDictionary)