webengine/osswebengine/WebCore/platform/network/symbian/CookieHandler.cpp
changeset 65 5bfc169077b2
parent 36 0ed94ceaa377
child 68 92a765b5b3e7
--- a/webengine/osswebengine/WebCore/platform/network/symbian/CookieHandler.cpp	Tue Feb 02 00:56:45 2010 +0200
+++ b/webengine/osswebengine/WebCore/platform/network/symbian/CookieHandler.cpp	Fri Mar 19 09:52:28 2010 +0200
@@ -19,7 +19,7 @@
 
 // INCLUDES
 #include "Cookiehandler.h"
-#include <cookiemanagerclient.h>
+#include <CookieManagerClient.h>
 
 // EXTERNAL DATA STRUCTURES
 
@@ -148,22 +148,10 @@
         popAndDestroyCount ++;
     }
     
-    // if the path is not specified then extract the domain name from the Url
-    // Why do we duplicate this code here?  It already exists in Cookie's constructl's call to adddefaultpathl?
-    TPtrC8 requestPath;
+    // if the domain is not specified then extract the domain name from the Url
     if(!cookieRecord.m_pathName.Length()) {
         const TDesC8& pathName= uriParser.Extract(EUriPath);
-        const TUint8 KCookiePathSeparator = '/';
-        requestPath.Set(pathName);
-        TInt sepPos = requestPath.LocateReverse( KCookiePathSeparator );
-        // if / is not the last character
-        if ( 0 <= sepPos && ( ( sepPos + 1 != requestPath.Length() ) ) )
-            {
-            // then remove characters after the right-most /
-            requestPath.Set( requestPath.Left( sepPos + 1 ) );
-            }
-
-        cookieRecord.m_pathName.Set(TPtrC(asciiToUnicodeLC(requestPath)->Des()));
+        cookieRecord.m_pathName.Set(TPtrC(asciiToUnicodeLC(pathName)->Des()));
         popAndDestroyCount ++;
     }
     
@@ -174,7 +162,6 @@
         cookieRecord);
     popAndDestroyCount +=6;
     // create a cookie
-    // Why are we using the stringpool only constructor of cookie?
     CCookie* cookie = CCookie::NewL( (*m_stringPool) );
     
     CleanupStack::PushL( cookie );