Changeset 143413 in webkit

Timestamp:

Feb 19, 2013, 4:55:49 PM (12 years ago)

Author:

dgrogan@chromium.org

Message:

Merge 142890

IndexedDB: Add a few more histogram calls
​https://bugs.webkit.org/show_bug.cgi?id=109762

Reviewed by Tony Chang.

A few places where commits could fail weren't being logged.

• Modules/indexeddb/IDBBackingStore.cpp:

(WebCore::IDBBackingStore::deleteDatabase):
(WebCore::IDBBackingStore::Transaction::commit):

TBR=​dgrogan@chromium.org
Review URL: ​https://codereview.chromium.org/12288067

File:

• branches/chromium/1410/Source/WebCore/Modules/indexeddb/IDBBackingStore.cpp (modified) (3 diffs)

branches/chromium/1410/Source/WebCore/Modules/indexeddb/IDBBackingStore.cpp

@@ -72 +72 @@
     GetNewVersionNumber,
     CreateIDBDatabaseMetaData,
+    DeleteDatabase,
+    TransactionCommit,
     IDBLevelDBBackingStoreInternalErrorMax,
 };
@@ -588 +590 @@
     transaction->remove(key);
 
-    return transaction->commit();
+    if (!transaction->commit()) {
+        INTERNAL_WRITE_ERROR(DeleteDatabase);
+        return false;
+    }
+    return true;
 }
 
@@ -1844 +1850 @@
     bool result = m_transaction->commit();
     m_transaction.clear();
+    if (!result)
+        INTERNAL_WRITE_ERROR(TransactionCommit);
     return result;
 }