Bug 17783 - Correct USE macros from CF to CFNETWORK
Summary: Correct USE macros from CF to CFNETWORK
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 523.x (Safari 3)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 17730
  Show dependency treegraph
 
Reported: 2008-03-11 15:03 PDT by Brent Fulgham
Modified: 2008-03-11 16:34 PDT (History)
0 users

See Also:


Attachments
Patch to conditionalize based on CFNetwork (3.94 KB, patch)
2008-03-11 16:05 PDT, Brent Fulgham
aroben: review+
Details | Formatted Diff | Diff
Small revision to extend ChangeLog. (3.55 KB, patch)
2008-03-11 16:22 PDT, Brent Fulgham
aroben: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Brent Fulgham 2008-03-11 15:03:10 PDT
There are a handful of places where the use of CFNetwork-specific calls are protected by USE(CF).  For now, the Windows (Curl) build still uses CoreFoundation, so it would be nice if these were properly conditionalized.

The attached patch is based on changes http://bugs.webkit.org/show_bug.cgi?id=17730
Comment 1 Brent Fulgham 2008-03-11 16:05:15 PDT
Created attachment 19681 [details]
Patch to conditionalize based on CFNetwork

The attached patch:
* Uses "USE(CFNETWORK)" instead of "PLATFORM(CF)" where appropriate.
* Fixes a bug in the call to InternetGetCookie, which was previously passing in the new blank UChar vector, rather than the name of the cookie to retrieve.
Comment 2 Adam Roben (:aroben) 2008-03-11 16:10:35 PDT
Comment on attachment 19681 [details]
Patch to conditionalize based on CFNetwork

You should add a summary of the changes to the ChangeLog. Something like "Compilation fixes when using CoreFoundation but not CFNetwork".

r=me, though the ResourceHandle.h change seems out of place in this patch.
Comment 3 Brent Fulgham 2008-03-11 16:22:16 PDT
Created attachment 19685 [details]
Small revision to extend ChangeLog.

* Updated Changelog to reference bug and explain change.
* Removed "ResourceHandle.h" change.  This will be landed in a future patch.
Comment 4 Adam Roben (:aroben) 2008-03-11 16:23:42 PDT
Comment on attachment 19685 [details]
Small revision to extend ChangeLog.

r=me
Comment 5 Matt Lilek 2008-03-11 16:34:29 PDT
Committed revision 30972.