[Webkit-unassigned] [Bug 23092] Conditionalize CFNetwork-specific logic in WebKit.dll

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=23092


aroben at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #26516|review?                     |review+
               Flag|                            |




------- Comment #7 from aroben at apple.com  2009-01-09 11:11 PDT -------
(From update of attachment 26516)
> Index: WebKit/win/WebDownload.cpp
> ===================================================================
> --- WebKit/win/WebDownload.cpp	(revision 39682)
> +++ WebKit/win/WebDownload.cpp	(working copy)
> @@ -38,12 +38,16 @@
>  #include "WebURLCredential.h"
>  #include "WebURLResponse.h"
>  
> +#include <wtf/platform.h>

I'm surprised you need to include this. I thought config.h pulled it in. In any
case, "Platform" should be capitalized, and this header should be in ASCII
order with the others (unless there's a reason why it can't be, in which case a
comment would be good).

> +#if USE(CFNETWORK)
>  #include <WebCore/AuthenticationCF.h>
> +#endif

I'm surprised this #include is still needed in WebDownload.cpp after these
changes.

> +    static const WebCore::String BundleExtension;
> +    static const UInt32 BundleMagicNumber;

I think a slightly better approach would be to make some static getter
functions to return these values. The main advantage of this is that it won't
require the BundleExtension String to be constructed when the DLL is first
loaded (a "static initializer"). Whether or not you turn these into getter
functions, they should begin with lowercase letters.

r=me


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.