Title: [102937] trunk/Source/WebCore
Revision
102937
Author
[email protected]
Date
2011-12-15 07:53:48 -0800 (Thu, 15 Dec 2011)

Log Message

Removing BlackBerryCookieCache from the build system
https://bugs.webkit.org/show_bug.cgi?id=74318

Patch by Otto Cheung <[email protected]> on 2011-12-15
Reviewed by Rob Buis.

Removing any references to BlackBerryCookieCache in files that are upstreamed.

We are removing the BlackBerryCookieCache because the CookieManager cookie structure has been
refactored to have a similar structure to the cache. Since the read speeds in both structures
are similar, we no longer need the cache to speed up read performance.

No new tests, this patch by itself will not work. This patch is only removing
the references to BlackBerryCookieCache in files that are upstreamed.

* PlatformBlackBerry.cmake:
* platform/network/blackberry/NetworkJob.cpp:
(WebCore::NetworkJob::handleNotifyHeaderReceived):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (102936 => 102937)


--- trunk/Source/WebCore/ChangeLog	2011-12-15 15:36:15 UTC (rev 102936)
+++ trunk/Source/WebCore/ChangeLog	2011-12-15 15:53:48 UTC (rev 102937)
@@ -1,3 +1,23 @@
+2011-12-15  Otto Cheung  <[email protected]>
+
+        Removing BlackBerryCookieCache from the build system
+        https://bugs.webkit.org/show_bug.cgi?id=74318
+
+        Reviewed by Rob Buis.
+
+        Removing any references to BlackBerryCookieCache in files that are upstreamed.
+
+        We are removing the BlackBerryCookieCache because the CookieManager cookie structure has been
+        refactored to have a similar structure to the cache. Since the read speeds in both structures
+        are similar, we no longer need the cache to speed up read performance.
+
+        No new tests, this patch by itself will not work. This patch is only removing
+        the references to BlackBerryCookieCache in files that are upstreamed.
+
+        * PlatformBlackBerry.cmake:
+        * platform/network/blackberry/NetworkJob.cpp:
+        (WebCore::NetworkJob::handleNotifyHeaderReceived):
+
 2011-12-15  Kenneth Rohde Christiansen  <[email protected]>
 
         [Qt] Mobile theme improvements

Modified: trunk/Source/WebCore/PlatformBlackBerry.cmake (102936 => 102937)


--- trunk/Source/WebCore/PlatformBlackBerry.cmake	2011-12-15 15:36:15 UTC (rev 102936)
+++ trunk/Source/WebCore/PlatformBlackBerry.cmake	2011-12-15 15:53:48 UTC (rev 102937)
@@ -133,7 +133,6 @@
     page/blackberry/DragControllerBlackBerry.cpp
     page/blackberry/EventHandlerBlackBerry.cpp
     page/blackberry/FrameBlackBerry.cpp
-    platform/blackberry/BlackBerryCookieCache.cpp
     platform/blackberry/ClipboardBlackBerry.cpp
     platform/blackberry/ContextMenuBlackBerry.cpp
     platform/blackberry/ContextMenuItemBlackBerry.cpp

Modified: trunk/Source/WebCore/platform/network/blackberry/NetworkJob.cpp (102936 => 102937)


--- trunk/Source/WebCore/platform/network/blackberry/NetworkJob.cpp	2011-12-15 15:36:15 UTC (rev 102936)
+++ trunk/Source/WebCore/platform/network/blackberry/NetworkJob.cpp	2011-12-15 15:53:48 UTC (rev 102937)
@@ -21,7 +21,6 @@
 
 #include "AboutData.h"
 #include "Base64.h"
-#include "BlackBerryCookieCache.h"
 #include "CookieManager.h"
 #include "CredentialStorage.h"
 #include "Frame.h"
@@ -305,7 +304,6 @@
         if (m_frame && m_frame->loader() && m_frame->loader()->client()
             && static_cast<FrameLoaderClientBlackBerry*>(m_frame->loader()->client())->cookiesEnabled())
             handleSetCookieHeader(value);
-        BlackBerryCookieCache::instance().clearAllCookiesForHost(m_pageGroupName, m_response.url());
     }
 
     if (lowerKey == "www-authenticate")
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to