From: ggaren@apple.com <ggaren@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Wed, 6 Mar 2013 06:54:01 +0000 (+0000)
Subject: Autoreleased cached pages slow down the PLT by 2%
X-Git-Url: http://git.webkit.org/?p=WebKit-https.git;a=commitdiff_plain;h=7dabb8afb246a916131703a12fca74fb18a121c8

Autoreleased cached pages slow down the PLT by 2%
https://bugs.webkit.org/show_bug.cgi?id=111522

Reviewed by Filip Pizlo.

Let's stop doing that.

2% PLT speedup.

* history/PageCache.cpp:
(WebCore::PageCache::remove):


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

diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
index 3fa038c..e58e919 100644
--- a/Source/WebCore/ChangeLog
+++ b/Source/WebCore/ChangeLog
@@ -1,3 +1,17 @@
+2013-03-05  Geoffrey Garen  <ggaren@apple.com>
+
+        Autoreleased cached pages slow down the PLT by 2%
+        https://bugs.webkit.org/show_bug.cgi?id=111522
+
+        Reviewed by Filip Pizlo.
+
+        Let's stop doing that.
+
+        2% PLT speedup.
+
+        * history/PageCache.cpp:
+        (WebCore::PageCache::remove):
+
 2013-03-05  Hayato Ito  <hayato@chromium.org>
 
         [Shadow] Implements event re-targeting for Touch Events.
diff --git a/Source/WebCore/history/PageCache.cpp b/Source/WebCore/history/PageCache.cpp
index 8fae41b..54912a8 100644
--- a/Source/WebCore/history/PageCache.cpp
+++ b/Source/WebCore/history/PageCache.cpp
@@ -485,7 +485,7 @@ void PageCache::remove(HistoryItem* item)
     if (!item || !item->m_cachedPage)
         return;
 
-    autorelease(item->m_cachedPage.release());
+    item->m_cachedPage.clear();
     removeFromLRUList(item);
     --m_size;