[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-9427-gc2be6fc

andreip at google.com andreip at google.com
Wed Dec 22 13:44:03 UTC 2010


The following commit has been merged in the debian/experimental branch:
commit 073c3d61d957b820e1b34a9870685f81bcbad509
Author: andreip at google.com <andreip at google.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Fri Sep 24 15:23:35 2010 +0000

    2010-09-24  Andrei Popescu  <andreip at google.com>
    
            Reviewed by Jeremy Orlow.
    
            IDBTransactionCoordinator incorrectly assumes that aborted transactions must have been started or running.
            https://bugs.webkit.org/show_bug.cgi?id=46459
    
            No new tests, storage/indexeddb/transaction-basics.html covers this.
    
            Also fixes http://code.google.com/p/chromium/issues/detail?id=56732
    
            * storage/IDBTransactionCoordinator.cpp:
            (WebCore::IDBTransactionCoordinator::didFinishTransaction):
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@68254 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 265d608..b75acf8 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,17 @@
+2010-09-24  Andrei Popescu  <andreip at google.com>
+
+        Reviewed by Jeremy Orlow.
+
+        IDBTransactionCoordinator incorrectly assumes that aborted transactions must have been started or running.
+        https://bugs.webkit.org/show_bug.cgi?id=46459
+
+        No new tests, storage/indexeddb/transaction-basics.html covers this.
+
+        Also fixes http://code.google.com/p/chromium/issues/detail?id=56732
+
+        * storage/IDBTransactionCoordinator.cpp:
+        (WebCore::IDBTransactionCoordinator::didFinishTransaction):
+
 2010-09-24  Hans Wennborg  <hans at chromium.org>
 
         Reviewed by Steve Block.
diff --git a/WebCore/storage/IDBTransactionCoordinator.cpp b/WebCore/storage/IDBTransactionCoordinator.cpp
index 4b06371..0ece309 100644
--- a/WebCore/storage/IDBTransactionCoordinator.cpp
+++ b/WebCore/storage/IDBTransactionCoordinator.cpp
@@ -67,10 +67,9 @@ void IDBTransactionCoordinator::didFinishTransaction(IDBTransactionBackendImpl*
     if (m_startedTransactions.contains(transaction)) {
         ASSERT(!m_runningTransactions.contains(transaction));
         m_startedTransactions.remove(transaction);
-    } else {
-        ASSERT(m_runningTransactions.contains(transaction));
+    } else if (m_runningTransactions.contains(transaction))
         m_runningTransactions.remove(transaction);
-    }
+
     m_transactions.remove(transaction->id());
 
     processStartedTransactions();

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list