Follow-up to https://trac.webkit.org/r222239 to fix crashes.
authorap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 30 Sep 2017 02:11:20 +0000 (02:11 +0000)
committerap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 30 Sep 2017 02:11:20 +0000 (02:11 +0000)
rdar://problem/34705779

Addressing review comments that I somehow landed without.

* WebView/WebHTMLView.mm: Undo adding #if guards around -_web_uniqueWebDataURL

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

Source/WebKitLegacy/mac/ChangeLog
Source/WebKitLegacy/mac/WebView/WebHTMLView.mm

index 72a24f1..73a569b 100644 (file)
@@ -1,3 +1,12 @@
+2017-09-29  Alexey Proskuryakov  <ap@apple.com>
+
+        Follow-up to https://trac.webkit.org/r222239 to fix crashes.
+        rdar://problem/34705779
+
+        Addressing review comments that I somehow landed without.
+
+        * WebView/WebHTMLView.mm: Undo adding #if guards around -_web_uniqueWebDataURL
+
 2017-09-29  Chris Dumez  <cdumez@apple.com>
 
         Split some logic out of VisitedLinkStore and make it reusable
index 1625d07..ef31682 100644 (file)
@@ -7292,9 +7292,8 @@ static CGImageRef selectionImage(Frame* frame, bool forceBlackText)
 
 @end
 
-#if (PLATFORM(IOS) && __IPHONE_OS_VERSION_MIN_REQUIRED < 110000) || (PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED < 101300)
-// This is used by AppKit/TextKit and is implemented here in part so that WebDataProtocolScheme is only defined once.
-// FIXME: Really should have an @interface for this somewhere in this file or an include. Not sure why it compiles without one.
+// This is used by AppKit/TextKit. It should be possible to remove this once
+// -[NSAttributedString _documentFromRange:document:documentAttributes:subresources:] is removed.
 @implementation NSURL (WebDataURL)
 
 + (NSURL *)_web_uniqueWebDataURL
@@ -7303,7 +7302,6 @@ static CGImageRef selectionImage(Frame* frame, bool forceBlackText)
 }
 
 @end
-#endif
 
 #if PLATFORM(MAC)