Changeset 194420 in webkit

Timestamp:

Dec 25, 2015, 12:46:47 PM (11 years ago)

Author:

ap@apple.com

Message:

ASSERT(m_nsResponse) in ResourceResponse::platformCertificateInfo() when running tests in iOS simulator
​https://bugs.webkit.org/show_bug.cgi?id=152559

Reviewed by Andy Estes.

Fix a regression from r194378.

• platform/network/cocoa/ResourceResponseCocoa.mm:

(WebCore::ResourceResponse::platformCertificateInfo):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/network/cocoa/ResourceResponseCocoa.mm (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2015-12-25  Alexey Proskuryakov  <ap@apple.com>
+
+        ASSERT(m_nsResponse) in ResourceResponse::platformCertificateInfo() when running tests in iOS simulator
+        https://bugs.webkit.org/show_bug.cgi?id=152559
+
+        Reviewed by Andy Estes.
+
+        Fix a regression from r194378.
+
+        * platform/network/cocoa/ResourceResponseCocoa.mm:
+        (WebCore::ResourceResponse::platformCertificateInfo):
+
 2015-12-25  David Kilzer  <ddkilzer@apple.com>
 

trunk/Source/WebCore/platform/network/cocoa/ResourceResponseCocoa.mm

@@ -68 +68 @@
 CertificateInfo ResourceResponse::platformCertificateInfo() const
 {
+#if USE(CFNETWORK)
+    ASSERT(m_cfResponse);
+    CFURLResponseRef cfResponse = m_cfResponse.get();
+#else
     ASSERT(m_nsResponse);
-    auto cfResponse = [m_nsResponse _CFURLResponse];
+    CFURLResponseRef cfResponse = [m_nsResponse _CFURLResponse];
+#endif
+
     if (!cfResponse)
         return { };