Changeset 174156 in webkit

Timestamp:

Oct 1, 2014 7:31:35 AM (10 years ago)

Author:

mitz@apple.com

Message:

REGRESSION (r173423): [iOS] Sites with EV certificates appear as normal HTTPS sites
​https://bugs.webkit.org/show_bug.cgi?id=137262

Reviewed by Antti Koivisto.

No new tests, because the API test harness doesn’t run an HTTP server to test against.

• platform/network/cf/ResourceResponseCFNet.cpp:

(WebCore::ResourceResponse::platformCertificateInfo): Return a CertificateInfo initialized
with the certificate array from the response.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/network/cf/ResourceResponseCFNet.cpp (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2014-10-01  Dan Bernstein  <mitz@apple.com>
+
+        REGRESSION (r173423): [iOS] Sites with EV certificates appear as normal HTTPS sites
+        https://bugs.webkit.org/show_bug.cgi?id=137262
+
+        Reviewed by Antti Koivisto.
+
+        No new tests, because the API test harness doesn’t run an HTTP server to test against.
+
+        * platform/network/cf/ResourceResponseCFNet.cpp:
+        (WebCore::ResourceResponse::platformCertificateInfo): Return a CertificateInfo initialized
+        with the certificate array from the response.
+
 2014-09-30  Yusuke Suzuki  <utatane.tea@gmail.com>
 

trunk/Source/WebCore/platform/network/cf/ResourceResponseCFNet.cpp

@@ -31 +31 @@
 #include "HTTPParsers.h"
 #include "MIMETypeRegistry.h"
+#include "WebCoreSystemInterface.h"
 #include <CFNetwork/CFURLResponsePriv.h>
 #include <wtf/RetainPtr.h>
@@ -116 +117 @@
 CertificateInfo ResourceResponse::platformCertificateInfo() const
 {
+#if PLATFORM(COCOA)
+    return CertificateInfo(adoptCF(wkCopyNSURLResponseCertificateChain(nsURLResponse())));
+#endif
     return CertificateInfo();
 }