Roll out r51354, r51460, and r51461 to see if removing them will fix
authorbweinstein@apple.com <bweinstein@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 25 Nov 2009 07:04:44 +0000 (07:04 +0000)
committerbweinstein@apple.com <bweinstein@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 25 Nov 2009 07:04:44 +0000 (07:04 +0000)
the Windows test bots, which are currently not very happy.

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

22 files changed:
WebCore/ChangeLog
WebCore/WebCore.base.exp
WebCore/WebCore.vcproj/WebCore.vcproj
WebCore/WebCore.xcodeproj/project.pbxproj
WebCore/bindings/js/JSWebSocketCustom.cpp
WebCore/platform/mac/WebCoreSystemInterface.h
WebCore/platform/mac/WebCoreSystemInterface.mm
WebCore/platform/network/CredentialStorage.h
WebCore/platform/network/cf/CredentialStorageCFNet.cpp [deleted file]
WebCore/platform/network/cf/SocketStreamHandle.h
WebCore/platform/network/cf/SocketStreamHandleCFNet.cpp
WebCore/platform/network/mac/CredentialStorageMac.mm [deleted file]
WebKit/mac/ChangeLog
WebKit/mac/WebCoreSupport/WebSystemInterface.m
WebKitLibraries/ChangeLog
WebKitLibraries/WebKitSystemInterface.h
WebKitLibraries/libWebKitSystemInterfaceLeopard.a
WebKitLibraries/libWebKitSystemInterfaceSnowLeopard.a
WebKitLibraries/libWebKitSystemInterfaceTiger.a
WebKitLibraries/win/include/WebKitSystemInterface/WebKitSystemInterface.h
WebKitLibraries/win/lib/WebKitSystemInterface.lib
WebKitLibraries/win/lib/WebKitSystemInterface_debug.lib

index feecb4aab978901f898d07b59152ae5753c70e17..32d6d9bffaa2cabf7ac3797d73d42515ec960336 100644 (file)
         * rendering/RenderThemeWin.cpp:
         * rendering/RenderThemeWin.h:
 
-2009-11-24  Alexey Proskuryakov  <ap@apple.com>
-
-        Reviewed by Brady Eidson.
-
-        https://bugs.webkit.org/show_bug.cgi?id=31844
-        SocketStreamHandleCFNet should support CONNECT proxy credentials
-
-        * WebCore.vcproj/WebCore.vcproj:
-        * WebCore.xcodeproj/project.pbxproj:
-        * platform/network/CredentialStorage.h:
-        * platform/network/mac/CredentialStorageMac.mm: Added.
-        (WebCore::CredentialStorage::getFromPersistentStorage):
-        * platform/network/cf/CredentialStorageCFNet.cpp: Added.
-        (WebCore::CredentialStorage::getFromPersistentStorage):
-        Add support for fetching credentials from persistent storage (CFNet version is currently
-        a stub).
-
-        * bindings/js/JSWebSocketCustom.cpp: Removed an unneeded include.
-
-        * platform/network/cf/SocketStreamHandle.h:
-        * platform/network/cf/SocketStreamHandleCFNet.cpp:
-        (WebCore::SocketStreamHandle::SocketStreamHandle):
-        (WebCore::SocketStreamHandle::createStreams):
-        (WebCore::getStoredCONNECTProxyCredentials):
-        (WebCore::authenticationSchemeFromAuthenticationMethod):
-        (WebCore::SocketStreamHandle::addCONNECTCredentials):
-        (WebCore::SocketStreamHandle::readStreamCallback):
-        Check if connection attempt was resulted in 407, and try stored credentials if it did.
-
-        * platform/mac/WebCoreSystemInterface.h:
-        * platform/mac/WebCoreSystemInterface.mm:
-        * WebCore.base.exp:
-        Updated WKSI.
-
 2009-11-24  Simon Fraser  <simon.fraser@apple.com>
 
         Reviewed by Dan Bernstein.
index 6aa2058f2dc844893b10bc4512515422a104f9e5..79f665abfe56528e647fa4bb8e92e751b2bd1598 100644 (file)
@@ -993,7 +993,6 @@ _stringIsCaseInsensitiveEqualToString
 _suggestedFilenameWithMIMEType
 _wkAdvanceDefaultButtonPulseAnimation
 _wkCGContextGetShouldSmoothFonts
-_wkCopyCONNECTProxyResponse
 _wkCreateCustomCFReadStream
 _wkCreateNSURLConnectionDelegateProxy
 _wkCreateURLNPasteboardFlavorTypeName
@@ -1032,8 +1031,6 @@ _wkQTMovieMaxTimeSeekable
 _wkQTMovieSetShowClosedCaptions
 _wkQTMovieViewSetDrawSynchronously
 _wkSetCGFontRenderingMode
-_wkSetCONNECTProxyAuthorizationForStream
-_wkSetCONNECTProxyForStream
 _wkSetDragImage
 _wkSetNSURLConnectionDefersCallbacks
 _wkSetNSURLRequestShouldContentSniff
index be65836b0928fd130e9cb7527f2cfcb207e6accb..dc44895b7ac4b2a2696c9e0d65c31403982d321d 100644 (file)
                                                        />\r
                                                </FileConfiguration>\r
                                        </File>\r
-                                       <File\r
-                                               RelativePath="..\platform\network\cf\CredentialStorageCFNet.cpp"\r
-                                               >\r
-                                               <FileConfiguration\r
-                                                       Name="Debug_Cairo|Win32"\r
-                                                       ExcludedFromBuild="true"\r
-                                                       >\r
-                                                       <Tool\r
-                                                               Name="VCCLCompilerTool"\r
-                                                       />\r
-                                               </FileConfiguration>\r
-                                               <FileConfiguration\r
-                                                       Name="Release_Cairo|Win32"\r
-                                                       ExcludedFromBuild="true"\r
-                                                       >\r
-                                                       <Tool\r
-                                                               Name="VCCLCompilerTool"\r
-                                                       />\r
-                                               </FileConfiguration>\r
-                                       </File>\r
                                        <File\r
                                                RelativePath="..\platform\network\cf\DNSCFNet.cpp"\r
                                                >\r
index 981b35b24b9938f74d397ae8671cab906cc08884..c300ee1a2ca799e66b308cae13ac1660af23ed58 100644 (file)
                E1ADEDD50E76BD60004A1A5E /* DOMMessagePortInternal.h in Headers */ = {isa = PBXBuildFile; fileRef = E1ADEDD40E76BD60004A1A5E /* DOMMessagePortInternal.h */; };
                E1ADEDDA0E76BD93004A1A5E /* JSMessagePort.h in Headers */ = {isa = PBXBuildFile; fileRef = E1ADEDD80E76BD93004A1A5E /* JSMessagePort.h */; };
                E1ADEDDB0E76BD93004A1A5E /* JSMessagePort.cpp in Sources */ = {isa = PBXBuildFile; fileRef = E1ADEDD90E76BD93004A1A5E /* JSMessagePort.cpp */; };
-               E1B4CD2510B322E200BFFD7E /* CredentialStorageMac.mm in Sources */ = {isa = PBXBuildFile; fileRef = E1B4CD2410B322E200BFFD7E /* CredentialStorageMac.mm */; };
                E1BE512D0CF6C512002EA959 /* XSLTUnicodeSort.cpp in Sources */ = {isa = PBXBuildFile; fileRef = E1BE512B0CF6C512002EA959 /* XSLTUnicodeSort.cpp */; };
                E1BE512E0CF6C512002EA959 /* XSLTUnicodeSort.h in Headers */ = {isa = PBXBuildFile; fileRef = E1BE512C0CF6C512002EA959 /* XSLTUnicodeSort.h */; };
                E1C362EF0EAF2AA9007410BC /* JSWorkerLocation.h in Headers */ = {isa = PBXBuildFile; fileRef = E1C362ED0EAF2AA9007410BC /* JSWorkerLocation.h */; };
                E1ADEDD40E76BD60004A1A5E /* DOMMessagePortInternal.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = DOMMessagePortInternal.h; sourceTree = "<group>"; };
                E1ADEDD80E76BD93004A1A5E /* JSMessagePort.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = JSMessagePort.h; sourceTree = "<group>"; };
                E1ADEDD90E76BD93004A1A5E /* JSMessagePort.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JSMessagePort.cpp; sourceTree = "<group>"; };
-               E1B4CD2410B322E200BFFD7E /* CredentialStorageMac.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = CredentialStorageMac.mm; sourceTree = "<group>"; };
                E1BE512B0CF6C512002EA959 /* XSLTUnicodeSort.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = XSLTUnicodeSort.cpp; sourceTree = "<group>"; };
                E1BE512C0CF6C512002EA959 /* XSLTUnicodeSort.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = XSLTUnicodeSort.h; sourceTree = "<group>"; };
                E1C362ED0EAF2AA9007410BC /* JSWorkerLocation.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = JSWorkerLocation.h; sourceTree = "<group>"; };
                                934F71430D5A6F5300018D69 /* AuthenticationChallenge.h */,
                                514C76420CE9234E007EF3CD /* AuthenticationMac.h */,
                                514C76430CE9234E007EF3CD /* AuthenticationMac.mm */,
-                               E1B4CD2410B322E200BFFD7E /* CredentialStorageMac.mm */,
                                514C76440CE9234E007EF3CD /* FormDataStreamMac.h */,
                                514C76450CE9234E007EF3CD /* FormDataStreamMac.mm */,
                                1A7FA6470DDA3CBA0028F8A5 /* NetworkStateNotifierMac.cpp */,
                                18C5FCA510A3991F0048438D /* WebKitSharedScript.cpp in Sources */,
                                18C5FCA810A3991F0048438D /* WebKitSharedScriptRepository.cpp in Sources */,
                                18C5FCB710A3C6F20048438D /* SharedScriptContext.cpp in Sources */,
-                               E1B4CD2510B322E200BFFD7E /* CredentialStorageMac.mm in Sources */,
                                6E47E66010B7944B00B186C8 /* WebGLGetInfo.cpp in Sources */,
                        );
                        runOnlyForDeploymentPostprocessing = 0;
index 238b041d597aae474b6642e0856891d3be6da01d..33c3fcd58624993f15f88354c7c13acd4c9241fc 100644 (file)
@@ -38,6 +38,7 @@
 #include "KURL.h"
 #include "JSEventListener.h"
 #include "WebSocket.h"
+#include "NotImplemented.h"
 #include <runtime/Error.h>
 
 using namespace JSC;
index 917ab0b5b1da820231e7f03faa8e154bff0204ac..285a8bfef62f8c7614a5a6c656afb93f4dbe23b3 100644 (file)
@@ -139,9 +139,6 @@ extern void (*wkSignalCFReadStreamEnd)(CFReadStreamRef stream);
 extern void (*wkSignalCFReadStreamError)(CFReadStreamRef stream, CFStreamError *error);
 extern void (*wkSignalCFReadStreamHasBytes)(CFReadStreamRef stream);
 extern unsigned (*wkInitializeMaximumHTTPConnectionCountPerHost)(unsigned preferredConnectionCount);
-extern void (*wkSetCONNECTProxyForStream)(CFReadStreamRef, CFStringRef proxyHost, CFNumberRef proxyPort);
-extern void (*wkSetCONNECTProxyAuthorizationForStream)(CFReadStreamRef, CFStringRef proxyAuthorizationString);
-extern CFHTTPMessageRef (*wkCopyCONNECTProxyResponse)(CFReadStreamRef, CFURLRef responseURL);
 extern BOOL (*wkIsLatchingWheelEvent)(NSEvent *);
 
 #ifndef BUILDING_ON_TIGER
index 3a9c0114a6e8796e39b2a4c63b0c4293581ecfd3..2663b8d0f580593c44366da8a60bee6ea014e20c 100644 (file)
@@ -83,9 +83,6 @@ void (*wkSetNSURLConnectionDefersCallbacks)(NSURLConnection *, BOOL);
 void (*wkSetNSURLRequestShouldContentSniff)(NSMutableURLRequest *, BOOL);
 id (*wkCreateNSURLConnectionDelegateProxy)(void);
 unsigned (*wkInitializeMaximumHTTPConnectionCountPerHost)(unsigned preferredConnectionCount);
-void (*wkSetCONNECTProxyForStream)(CFReadStreamRef, CFStringRef proxyHost, CFNumberRef proxyPort);
-void (*wkSetCONNECTProxyAuthorizationForStream)(CFReadStreamRef, CFStringRef proxyAuthorizationString);
-CFHTTPMessageRef (*wkCopyCONNECTProxyResponse)(CFReadStreamRef, CFURLRef responseURL);
 BOOL (*wkIsLatchingWheelEvent)(NSEvent *);
 
 #ifndef BUILDING_ON_TIGER
index 21fcbadbbb08815b4a9d9c585db28870514654a7..5086f69f65882650dbb93f341b882b1eb897913d 100644 (file)
@@ -34,13 +34,9 @@ class ProtectionSpace;
 
 class CredentialStorage {
 public:
-    // WebCore session credential storage.
     static void set(const Credential&, const ProtectionSpace&, const KURL&);
     static Credential get(const ProtectionSpace&);
 
-    // OS persistent storage.
-    static Credential getFromPersistentStorage(const ProtectionSpace&);
-
     // These methods work for authentication schemes that support sending credentials without waiting for a request. E.g., for HTTP Basic authentication scheme
     // a client should assume that all paths at or deeper than the depth of a known protected resource share are within the same protection space.
     static bool set(const Credential&, const KURL&); // Returns true if the URL corresponds to a known protection space, so credentials could be updated.
diff --git a/WebCore/platform/network/cf/CredentialStorageCFNet.cpp b/WebCore/platform/network/cf/CredentialStorageCFNet.cpp
deleted file mode 100644 (file)
index ef7a501..0000000
+++ /dev/null
@@ -1,42 +0,0 @@
-/*
- * Copyright (C) 2009 Apple Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#include "config.h"
-#include "CredentialStorage.h"
-
-#include "Credential.h"
-#include "NotImplemented.h"
-#include "ProtectionSpace.h"
-
-namespace WebCore {
-
-Credential CredentialStorage::getFromPersistentStorage(const ProtectionSpace& protectionSpace)
-{
-    UNUSED_PARAM(protectionSpace);
-    notImplemented();
-    return Credential();
-}
-
-} // namespace WebCore
index 63bf9a70cb0f3975053ad930a12ede9d4afab29a..4cd10ca571e6b906e177a74bd4b034ec5f2c473c 100644 (file)
@@ -71,8 +71,6 @@ namespace WebCore {
 
         bool shouldUseSSL() const { return m_url.protocolIs("wss"); }
 
-        void addCONNECTCredentials(CFHTTPMessageRef response);
-
         static CFStringRef copyCFStreamDescription(void* );
         static void readStreamCallback(CFReadStreamRef, CFStreamEventType, void*);
         static void writeStreamCallback(CFWriteStreamRef, CFStreamEventType, void*);
@@ -100,7 +98,6 @@ namespace WebCore {
         RetainPtr<CFNumberRef> m_proxyPort;
 
         RetainPtr<CFHTTPMessageRef> m_proxyResponseMessage;
-        bool m_sentStoredCredentials;
         RetainPtr<CFReadStreamRef> m_readStream;
         RetainPtr<CFWriteStreamRef> m_writeStream;
 
index 4e64bbd0e08b2cbab9dddaca0ae2200118de44aa..8c6ff5698826997769a3d759ff158d421735d7e3 100644 (file)
 #include "config.h"
 #include "SocketStreamHandle.h"
 
-#include "Credential.h"
-#include "CredentialStorage.h"
 #include "Logging.h"
-#include "ProtectionSpace.h"
 #include "SocketStreamError.h"
 #include "SocketStreamHandleClient.h"
 #include <wtf/MainThread.h>
 
 #if PLATFORM(WIN)
 #include "LoaderRunLoopCF.h"
-#include <WebKitSystemInterface/WebKitSystemInterface.h>
-#else
-#include "WebCoreSystemInterface.h"
 #endif
 
 #ifdef BUILDING_ON_TIGER
 #define CFN_EXPORT extern
 #endif
 
+extern "C" {
+CFN_EXPORT const CFStringRef kCFStreamPropertyCONNECTProxy;
+CFN_EXPORT const CFStringRef kCFStreamPropertyCONNECTProxyHost;
+CFN_EXPORT const CFStringRef kCFStreamPropertyCONNECTProxyPort;
+}
+
 namespace WebCore {
 
 SocketStreamHandle::SocketStreamHandle(const KURL& url, SocketStreamHandleClient* client)
     : SocketStreamHandleBase(url, client)
     , m_connectingSubstate(New)
     , m_connectionType(Unknown)
-    , m_sentStoredCredentials(false)
 {
     LOG(Network, "SocketStreamHandle %p new client %p", this, m_client);
 
@@ -334,9 +333,13 @@ void SocketStreamHandle::createStreams()
         CFReadStreamSetProperty(m_readStream.get(), kCFStreamPropertySOCKSProxy, connectDictionary.get());
         break;
         }
-    case CONNECTProxy:
-        wkSetCONNECTProxyForStream(m_readStream.get(), m_proxyHost.get(), m_proxyPort.get());
+    case CONNECTProxy: {
+        const void* proxyKeys[] = { kCFStreamPropertyCONNECTProxyHost, kCFStreamPropertyCONNECTProxyPort };
+        const void* proxyValues[] = { m_proxyHost.get(), m_proxyPort.get() };
+        RetainPtr<CFDictionaryRef> connectDictionary(AdoptCF, CFDictionaryCreate(0, proxyKeys, proxyValues, sizeof(proxyKeys) / sizeof(*proxyKeys), &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks));
+        CFReadStreamSetProperty(m_readStream.get(), kCFStreamPropertyCONNECTProxy, connectDictionary.get());
         break;
+        }
     }
 
     if (shouldUseSSL()) {
@@ -348,84 +351,6 @@ void SocketStreamHandle::createStreams()
     }
 }
 
-static bool getStoredCONNECTProxyCredentials(const ProtectionSpace& protectionSpace, String& login, String& password)
-{
-    // Try system credential storage first, matching HTTP behavior (CFNetwork only asks the client for password if it couldn't find it in Keychain).
-    Credential storedCredential = CredentialStorage::getFromPersistentStorage(protectionSpace);
-    if (storedCredential.isEmpty())
-        storedCredential = CredentialStorage::get(protectionSpace);
-
-    if (storedCredential.isEmpty())
-        return false;
-
-    login = storedCredential.user();
-    password = storedCredential.password();
-
-    return true;
-}
-
-static ProtectionSpaceAuthenticationScheme authenticationSchemeFromAuthenticationMethod(CFStringRef method)
-{
-    if (CFEqual(method, kCFHTTPAuthenticationSchemeBasic))
-        return ProtectionSpaceAuthenticationSchemeHTTPBasic;
-    if (CFEqual(method, kCFHTTPAuthenticationSchemeDigest))
-        return ProtectionSpaceAuthenticationSchemeHTTPDigest;
-#ifndef BUILDING_ON_TIGER
-    if (CFEqual(method, kCFHTTPAuthenticationSchemeNTLM))
-        return ProtectionSpaceAuthenticationSchemeNTLM;
-    if (CFEqual(method, kCFHTTPAuthenticationSchemeNegotiate))
-        return ProtectionSpaceAuthenticationSchemeNegotiate;
-#endif
-    ASSERT_NOT_REACHED();
-    return ProtectionSpaceAuthenticationSchemeDefault;
-}
-
-void SocketStreamHandle::addCONNECTCredentials(CFHTTPMessageRef proxyResponse)
-{
-    RetainPtr<CFHTTPAuthenticationRef> authentication(AdoptCF, CFHTTPAuthenticationCreateFromResponse(0, proxyResponse));
-
-    if (!CFHTTPAuthenticationRequiresUserNameAndPassword(authentication.get())) {
-        // That's all we can offer...
-        m_client->didFail(this, SocketStreamError()); // FIXME: Provide a sensible error.
-        return;
-    }
-
-    int port = 0;
-    CFNumberGetValue(m_proxyPort.get(), kCFNumberIntType, &port);
-    RetainPtr<CFStringRef> methodCF(AdoptCF, CFHTTPAuthenticationCopyMethod(authentication.get()));
-    RetainPtr<CFStringRef> realmCF(AdoptCF, CFHTTPAuthenticationCopyRealm(authentication.get()));
-    ProtectionSpace protectionSpace(String(m_proxyHost.get()), port, ProtectionSpaceProxyHTTPS, String(realmCF.get()), authenticationSchemeFromAuthenticationMethod(methodCF.get()));
-    String login;
-    String password;
-    if (!m_sentStoredCredentials && getStoredCONNECTProxyCredentials(protectionSpace, login, password)) {
-        // Try to apply stored credentials, if we haven't tried those already.
-        RetainPtr<CFStringRef> loginCF(AdoptCF, login.createCFString());
-        RetainPtr<CFStringRef> passwordCF(AdoptCF, password.createCFString());
-        // Creating a temporary request to make CFNetwork apply credentials to it. Unfortunately, this cannot work with NTLM authentication.
-        RetainPtr<CFHTTPMessageRef> dummyRequest(AdoptCF, CFHTTPMessageCreateRequest(0, CFSTR("GET"), m_httpsURL.get(), kCFHTTPVersion1_1));
-
-        Boolean appliedCredentials = CFHTTPMessageApplyCredentials(dummyRequest.get(), authentication.get(), loginCF.get(), passwordCF.get(), 0);
-        ASSERT_UNUSED(appliedCredentials, appliedCredentials);
-
-        RetainPtr<CFStringRef> proxyAuthorizationString(AdoptCF, CFHTTPMessageCopyHeaderFieldValue(dummyRequest.get(), CFSTR("Proxy-Authorization")));
-
-        if (!proxyAuthorizationString) {
-            // Fails e.g. for NTLM auth.
-            m_client->didFail(this, SocketStreamError()); // FIXME: Provide a sensible error.
-            return;
-        }
-
-        // Setting the authorization results in a new connection attempt.
-        wkSetCONNECTProxyAuthorizationForStream(m_readStream.get(), proxyAuthorizationString.get());
-        m_sentStoredCredentials = true;
-        return;
-    }
-
-    // FIXME: Ask the client if credentials could not be found.
-
-    m_client->didFail(this, SocketStreamError()); // FIXME: Provide a sensible error.
-}
-
 CFStringRef SocketStreamHandle::copyCFStreamDescription(void* info)
 {
     SocketStreamHandle* handle = static_cast<SocketStreamHandle*>(info);
@@ -487,13 +412,7 @@ void SocketStreamHandle::readStreamCallback(CFStreamEventType type)
         break;
     case kCFStreamEventHasBytesAvailable: {
         if (m_connectingSubstate == WaitingForConnect) {
-            if (m_connectionType == CONNECTProxy) {
-                RetainPtr<CFHTTPMessageRef> proxyResponse(AdoptCF, wkCopyCONNECTProxyResponse(m_readStream.get(), m_httpsURL.get()));
-                if (proxyResponse && (407 == CFHTTPMessageGetResponseStatusCode(proxyResponse.get()))) {
-                    addCONNECTCredentials(proxyResponse.get());
-                    return;
-                }
-            }
+            // FIXME: Handle CONNECT proxy credentials here.
         } else if (m_connectingSubstate == WaitingForCredentials)
             break;
 
diff --git a/WebCore/platform/network/mac/CredentialStorageMac.mm b/WebCore/platform/network/mac/CredentialStorageMac.mm
deleted file mode 100644 (file)
index 66e94e9..0000000
+++ /dev/null
@@ -1,40 +0,0 @@
-/*
- * Copyright (C) 2009 Apple Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#include "config.h"
-#include "CredentialStorage.h"
-
-#include "AuthenticationMac.h"
-#include "Credential.h"
-
-namespace WebCore {
-
-Credential CredentialStorage::getFromPersistentStorage(const ProtectionSpace& protectionSpace)
-{
-    NSURLCredential *credential = [[NSURLCredentialStorage sharedCredentialStorage] defaultCredentialForProtectionSpace:mac(protectionSpace)];
-    return credential ? core(credential) : Credential();
-}
-
-} // namespace WebCore
index 3eff91b5045f3ebca31c8e73b6b5bec42e0e5bba..78a285da091dfe600026ddc9751789d75df276aa 100644 (file)
@@ -7,15 +7,6 @@
 
         * Configurations/FeatureDefines.xcconfig:
 
-2009-11-24  Alexey Proskuryakov  <ap@apple.com>
-
-        Reviewed by Brady Eidson.
-
-        https://bugs.webkit.org/show_bug.cgi?id=31844
-        SocketStreamHandleCFNet should support CONNECT proxy credentials
-
-        * WebCoreSupport/WebSystemInterface.m: (InitWebCoreSystemInterface): Add WKSI methods.
-
 2009-11-23  Simon Fraser  <simon.fraser@apple.com>
 
         Reviewed by Dan Bernstein.
index f2a215def979c8a3981e5d6336ce7cf439cdddba..56e01687a69d508b9f4c958843944fa2573374d6 100644 (file)
@@ -41,7 +41,6 @@ void InitWebCoreSystemInterface(void)
 
     INIT(AdvanceDefaultButtonPulseAnimation);
     INIT(CGContextGetShouldSmoothFonts);
-    INIT(CopyCONNECTProxyResponse);
     INIT(CreateCustomCFReadStream);
     INIT(CreateNSURLConnectionDelegateProxy);
     INIT(DrawCapsLockIndicator);
@@ -66,8 +65,6 @@ void InitWebCoreSystemInterface(void)
     INIT(MediaControllerThemeAvailable);
     INIT(PopupMenu);
     INIT(SetCGFontRenderingMode);
-    INIT(SetCONNECTProxyAuthorizationForStream);
-    INIT(SetCONNECTProxyForStream);
     INIT(SetDragImage);
     INIT(SetNSURLConnectionDefersCallbacks);
     INIT(SetNSURLRequestShouldContentSniff);
index b9a4fbdd4cbe402267d5dd63304734b9ce80f554..b9c5b121d1ff206d1836cf2cf6bfdcaa6e4642f6 100644 (file)
@@ -1,19 +1,3 @@
-2009-11-24  Alexey Proskuryakov  <ap@apple.com>
-
-        Reviewed by Brady Eidson.
-
-        https://bugs.webkit.org/show_bug.cgi?id=31844
-        SocketStreamHandleCFNet should support CONNECT proxy credentials
-
-        * WebKitSystemInterface.h:
-        * libWebKitSystemInterfaceLeopard.a:
-        * libWebKitSystemInterfaceSnowLeopard.a:
-        * libWebKitSystemInterfaceTiger.a:
-        * win/include/WebKitSystemInterface/WebKitSystemInterface.h:
-        * win/lib/WebKitSystemInterface.lib:
-        * win/lib/WebKitSystemInterface_debug.lib:
-        Update WebKitSystemInterface.
-
 2009-11-22  Dan Bernstein  <mitz@apple.com>
 
         Reviewed by Eric Carlson.
index f6234fe59cd61a6ee2db01e257a2419f7845bf0d..f1bc5231d66ce977edaec41d78651f4ad82c905b 100644 (file)
@@ -279,10 +279,6 @@ void WKSetCAAnimationValueFunction(CAPropertyAnimation*, NSString* function);
 
 unsigned WKInitializeMaximumHTTPConnectionCountPerHost(unsigned preferredConnectionCount);
 
-void WKSetCONNECTProxyForStream(CFReadStreamRef, CFStringRef proxyHost, CFNumberRef proxyPort);
-void WKSetCONNECTProxyAuthorizationForStream(CFReadStreamRef, CFStringRef proxyAuthorizationString);
-CFHTTPMessageRef WKCopyCONNECTProxyResponse(CFReadStreamRef, CFURLRef responseURL);
-
 BOOL WKIsLatchingWheelEvent(NSEvent *);
 
 #ifndef BUILDING_ON_TIGER
index 71f4659ee6ae18bf11ad061d17969257635d50a1..a13a2602d5d272d52a2f99e8c1e45d4fe2370654 100644 (file)
Binary files a/WebKitLibraries/libWebKitSystemInterfaceLeopard.a and b/WebKitLibraries/libWebKitSystemInterfaceLeopard.a differ
index 6ecfb8b6ffa756044646e01626f0b651a86b0abf..6ecad1e62f09700d951e6ed5d2910617d650da2a 100644 (file)
Binary files a/WebKitLibraries/libWebKitSystemInterfaceSnowLeopard.a and b/WebKitLibraries/libWebKitSystemInterfaceSnowLeopard.a differ
index bd780bdd742991b505831f3370dafe7a6c755bee..0393e36c1a3542b2b19e074484b4f7dbc67c9bb3 100644 (file)
Binary files a/WebKitLibraries/libWebKitSystemInterfaceTiger.a and b/WebKitLibraries/libWebKitSystemInterfaceTiger.a differ
index 91e09031ef981a3de0d27fb91a16fc9ced8cc089..b5ce0977b5932fbef9e41d0436d11e131849248c 100644 (file)
@@ -42,11 +42,6 @@ typedef struct _CFURLResponse* CFURLResponseRef;
 typedef struct OpaqueCFHTTPCookieStorage*  CFHTTPCookieStorageRef;
 typedef struct _CFURLRequest* CFMutableURLRequestRef;
 typedef const struct _CFURLRequest* CFURLRequestRef;
-typedef struct _CFURLCredential* CFURLCredentialRef;
-typedef struct __CFHTTPMessage* CFHTTPMessageRef;
-typedef const struct __CFNumber* CFNumberRef;
-typedef struct __CFReadStream* CFReadStreamRef;
-typedef const struct __CFURL* CFURLRef;
 
 void wkSetFontSmoothingLevel(int type);
 int wkGetFontSmoothingLevel();
@@ -82,10 +77,6 @@ void wkCFURLRequestSetHTTPRequestBodyParts(CFMutableURLRequestRef, CFArrayRef bo
 
 unsigned wkInitializeMaximumHTTPConnectionCountPerHost(unsigned preferredConnectionCount);
 
-void wkSetCONNECTProxyForStream(CFReadStreamRef, CFStringRef proxyHost, CFNumberRef proxyPort);
-void wkSetCONNECTProxyAuthorizationForStream(CFReadStreamRef, CFStringRef proxyAuthorizationString);
-CFHTTPMessageRef wkCopyCONNECTProxyResponse(CFReadStreamRef, CFURLRef responseURL);
-
 CFStringRef wkCFNetworkErrorGetLocalizedDescription(CFIndex errorCode);
 
 #endif // WebKitSystemInterface_h
index 25c1332b908f0f2c692f6baf74de21bb9b795dcc..d4f9dd9b0b437cdeeedac9ef8bb165a7c4635d34 100644 (file)
Binary files a/WebKitLibraries/win/lib/WebKitSystemInterface.lib and b/WebKitLibraries/win/lib/WebKitSystemInterface.lib differ
index 236bc9856b594ccc731db0db8fe266d23504f7e3..6d1858808b08c05b7a6ea5a65157276262b75ad9 100644 (file)
Binary files a/WebKitLibraries/win/lib/WebKitSystemInterface_debug.lib and b/WebKitLibraries/win/lib/WebKitSystemInterface_debug.lib differ