Title: [223108] branches/safari-604-branch/Source/WebCore
Revision
223108
Author
[email protected]
Date
2017-10-09 17:08:49 -0700 (Mon, 09 Oct 2017)

Log Message

Cherry-pick r222786. rdar://problem/34820881

Modified Paths

Diff

Modified: branches/safari-604-branch/Source/WebCore/ChangeLog (223107 => 223108)


--- branches/safari-604-branch/Source/WebCore/ChangeLog	2017-10-10 00:08:47 UTC (rev 223107)
+++ branches/safari-604-branch/Source/WebCore/ChangeLog	2017-10-10 00:08:49 UTC (rev 223108)
@@ -1,5 +1,17 @@
 2017-10-09  Jason Marcell  <[email protected]>
 
+        Cherry-pick r222786. rdar://problem/34820881
+
+    2017-10-03  Brent Fulgham  <[email protected]>
+
+            Unreviewed test fix after r222779.
+
+            * testing/cocoa/WebArchiveDumpSupport.mm:
+            (WebCoreTestSupport::createCFURLResponseFromResponseData): Setting NSSecureCoding in this test code should be
+            done based on build system, just like it is in the actual WebArchive handling code.
+
+2017-10-09  Jason Marcell  <[email protected]>
+
         Cherry-pick r222779. rdar://problem/34820881
 
     2017-10-03  Brent Fulgham  <[email protected]>

Modified: branches/safari-604-branch/Source/WebCore/testing/cocoa/WebArchiveDumpSupport.mm (223107 => 223108)


--- branches/safari-604-branch/Source/WebCore/testing/cocoa/WebArchiveDumpSupport.mm	2017-10-10 00:08:47 UTC (rev 223107)
+++ branches/safari-604-branch/Source/WebCore/testing/cocoa/WebArchiveDumpSupport.mm	2017-10-10 00:08:49 UTC (rev 223108)
@@ -41,13 +41,14 @@
 static CFURLResponseRef createCFURLResponseFromResponseData(CFDataRef responseData)
 {
     RetainPtr<NSKeyedUnarchiver> unarchiver = adoptNS([[NSKeyedUnarchiver alloc] initForReadingWithData:(NSData *)responseData]);
+    NSURLResponse *response;
+#if (PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED >= 101300) || PLATFORM(IOS)
+    // Because of <rdar://problem/34063313> we can't use this for decoding in older OS's.
     [unarchiver setRequiresSecureCoding:YES];
-    NSURLResponse *response;
     @try {
-#if (PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED >= 101300) || PLATFORM(IOS)
-        // Because of <rdar://problem/34063313> we can't use this for decoding in older OS's.
         response = [unarchiver decodeObjectOfClass:[NSURLResponse class] forKey:@"WebResourceResponse"]; // WebResourceResponseKey in WebResource.m
 #else
+    @try {
         response = [unarchiver decodeObjectForKey:@"WebResourceResponse"]; // WebResourceResponseKey in WebResource.m
 #endif
         [unarchiver finishDecoding];
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to