[SCM] WebKit Debian packaging branch, debian/unstable, updated. debian/1.1.15-1-40151-g37bb677

trey trey at 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Sat Sep 26 07:14:31 UTC 2009


The following commit has been merged in the debian/unstable branch:
commit 3192ec999fb20364f5467e3e565dd5b2bb1ea8d6
Author: trey <trey at 268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Fri Dec 13 02:29:47 2002 +0000

    	3117101 - PLT slows down as history fills up
    
    	I saw a 3% sloth effect from a huge (100k) history.  This change cut that cost in half.
    	I think we still have a marginally measurable cost for our worst practical case.
    
            Reviewed by rjw
    
            * History.subproj/WebHistoryPrivate.m:
            (-[WebHistoryPrivate insertEntry:atDateIndex:]):  Comment.
            (-[WebHistoryPrivate removeEntryForURLString:]):  Use removeIdentical
    	instead of remove, since we don't need to do all the equals comparisons.
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3031 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebKit/ChangeLog b/WebKit/ChangeLog
index de44d5a..288b165 100644
--- a/WebKit/ChangeLog
+++ b/WebKit/ChangeLog
@@ -1,3 +1,17 @@
+2002-12-12  Trey Matteson  <trey at apple.com>
+
+	3117101 - PLT slows down as history fills up
+
+	I saw a 3% sloth effect from a huge (100k) history.  This change cut that cost in half.
+	I think we still have a marginally measurable cost for our worst practical case.
+
+        Reviewed by rjw
+
+        * History.subproj/WebHistoryPrivate.m:
+        (-[WebHistoryPrivate insertEntry:atDateIndex:]):  Comment.
+        (-[WebHistoryPrivate removeEntryForURLString:]):  Use removeIdentical
+	instead of remove, since we don't need to do all the equals comparisons.
+
 2002-12-12  Richard Williamson   <rjw at apple.com>
 
         Changes to fix 3116584.
@@ -247,6 +261,7 @@
 2002-12-10  Trey Matteson  <trey at apple.com>
 
 	3092966 - going back goes to different page (can't go back to a POST page)
+	3123450 - if the user refuses a navigation, the b/f menus is wrong
 
 	We will rePOST data upon back/forward/refresh if our caches fail us.  The policy delegate
 	gets a crack at confirming this operation.  Latent bugs where the policy delegate was double
diff --git a/WebKit/History.subproj/WebHistoryPrivate.m b/WebKit/History.subproj/WebHistoryPrivate.m
index be1d9bd..6ed51d5 100644
--- a/WebKit/History.subproj/WebHistoryPrivate.m
+++ b/WebKit/History.subproj/WebHistoryPrivate.m
@@ -98,6 +98,7 @@
     entryDate = [entry lastVisitedDate];
     entriesForDate = [_entriesByDate objectAtIndex: dateIndex];
     count = [entriesForDate count];
+    // optimized for inserting oldest to youngest
     for (index = 0; index < count; ++index) {
         if ([entryDate compare: [[entriesForDate objectAtIndex: index] lastVisitedDate]] != NSOrderedAscending) {
             break;
@@ -126,7 +127,7 @@
     ASSERT(foundDate);
     
     entriesForDate = [_entriesByDate objectAtIndex: dateIndex];
-    [entriesForDate removeObject: entry];
+    [entriesForDate removeObjectIdenticalTo: entry];
 
     // remove this date entirely if there are no other entries on it
     if ([entriesForDate count] == 0) {

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list