Changeset 66005 in webkit

Timestamp:

Aug 25, 2010 4:36:54 AM (14 years ago)

Author:

commit-queue@webkit.org

Message:

2010-08-25 Yongjun Zhang <​yongjun_zhang@apple.com>

Reviewed by Alexey Proskuryakov.

​https://bugs.webkit.org/show_bug.cgi?id=44461
Assertion failure at WebCore/platform/network/CredentialStorage.cpp:85.

Remove the assertion at CredentialStorage.cpp:85 since it is legal to have
multiple forward slashes after the path component.

No test needed.

• platform/network/CredentialStorage.cpp: (WebCore::protectionSpaceMapKeyFromURL):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/network/CredentialStorage.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-08-25  Yongjun Zhang  <yongjun_zhang@apple.com>
+
+        Reviewed by Alexey Proskuryakov.
+
+        https://bugs.webkit.org/show_bug.cgi?id=44461
+        Assertion failure at WebCore/platform/network/CredentialStorage.cpp:85.
+
+        Remove the assertion at CredentialStorage.cpp:85 since it is legal to have
+        multiple forward slashes after the path component.
+
+        No test needed.
+
+        * platform/network/CredentialStorage.cpp:
+        (WebCore::protectionSpaceMapKeyFromURL):
+
 2010-08-25  Pavel Feldman  <pfeldman@chromium.org>
 

trunk/WebCore/platform/network/CredentialStorage.cpp

@@ -80 +80 @@
         directoryURL = directoryURL.substring(0, (index != directoryURLPathStart) ? index : directoryURLPathStart + 1);
     }
-    ASSERT(directoryURL.length() == directoryURLPathStart + 1 || directoryURL[directoryURL.length() - 1] != '/');
 
     return directoryURL;