RESOLVED FIXED Bug 189072
[CFNetwork] Update CFNetwork SPI use to use CFNetworkSPI.h more consistently 
https://bugs.webkit.org/show_bug.cgi?id=189072
Summary [CFNetwork] Update CFNetwork SPI use to use CFNetworkSPI.h more consistently
Darin Adler
Reported 2018-08-28 19:32:39 PDT
[CFNetwork] Update CFNetwork SPI use to use CFNetworkSPI.h more consistently
Attachments
Patch (16.60 KB, patch)
2018-08-28 19:42 PDT, Darin Adler 		no flags
DetailsFormatted DiffDiff
Patch (26.44 KB, patch)
2018-08-28 20:34 PDT, Darin Adler 		no flags
DetailsFormatted DiffDiff
Patch (26.22 KB, patch)
2018-08-28 21:10 PDT, Darin Adler 		no flags
DetailsFormatted DiffDiff
Patch (26.18 KB, patch)
2018-08-29 19:40 PDT, Darin Adler 		no flags
DetailsFormatted DiffDiff
Patch (26.48 KB, patch)
2018-09-01 13:58 PDT, Darin Adler 		no flags
DetailsFormatted DiffDiff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.
Darin Adler
Comment 1 2018-08-28 19:42:40 PDT
Created attachment 348380 [details] Patch
Darin Adler
Comment 2 2018-08-28 20:34:40 PDT
Created attachment 348383 [details] Patch
Darin Adler
Comment 3 2018-08-28 21:10:53 PDT
Created attachment 348387 [details] Patch
Darin Adler
Comment 4 2018-08-29 19:40:24 PDT
Created attachment 348471 [details] Patch
Darin Adler
Comment 5 2018-09-01 13:58:55 PDT
Created attachment 348716 [details] Patch
mitz
Comment 6 2018-09-01 18:08:09 PDT
Comment on attachment 348716 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=348716&action=review I’d r+ but I don’t see an r?. > Source/WebCore/PAL/pal/spi/cf/CFNetworkSPI.h:177 > +#if (PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED >= 101300) || (PLATFORM(IOS) && __IPHONE_OS_VERSION_MIN_REQUIRED >= 110000) Not new to this patch, but in SPI headers we should be using the SDK version rather than the deployment target, since the goal of the header is to reflect the contents of an internal SDK header.
Darin Adler
Comment 7 2018-09-01 19:26:31 PDT
Comment on attachment 348716 [details] Patch Just noticed this is finally building, so setting review? on it now.
Darin Adler
Comment 8 2018-09-01 19:28:16 PDT
(In reply to mitz from comment #6) > > Source/WebCore/PAL/pal/spi/cf/CFNetworkSPI.h:177 > > +#if (PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED >= 101300) || (PLATFORM(IOS) && __IPHONE_OS_VERSION_MIN_REQUIRED >= 110000) > > Not new to this patch, but in SPI headers we should be using the SDK version > rather than the deployment target, since the goal of the header is to > reflect the contents of an internal SDK header. That sounds exactly right to me, but I don’t understand it well enough to know exactly what to do. I would be happy to help fix it at any time, but I don’t want to risk breaking this patch by trying it as part of this one.
WebKit Commit Bot
Comment 9 2018-09-01 19:55:05 PDT
Comment on attachment 348716 [details] Patch Clearing flags on attachment: 348716 Committed r235585: <https://trac.webkit.org/changeset/235585>
WebKit Commit Bot
Comment 10 2018-09-01 19:55:07 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 11 2018-09-01 19:56:19 PDT
<rdar://problem/44038929>
David Kilzer (:ddkilzer)
Comment 12 2018-09-05 04:48:07 PDT
(In reply to WebKit Commit Bot from comment #9) > Comment on attachment 348716 [details] > Patch > > Clearing flags on attachment: 348716 > > Committed r235585: <https://trac.webkit.org/changeset/235585> Follow-up fix to remove references to WebDownloadInternal.h from the WebKitLegacy Xcode project: Committed r235663: <https://trac.webkit.org/changeset/235663>
Note You need to log in before you can comment on or make changes to this bug.