Improve IndexedDB's quota support

content/browser/in_process_webkit/indexed_db_transaction_callbacks.cc

Index: content/browser/in_process_webkit/indexed_db_transaction_callbacks.cc
diff --git a/content/browser/in_process_webkit/indexed_db_transaction_callbacks.cc b/content/browser/in_process_webkit/indexed_db_transaction_callbacks.cc
index c8062421f93c7778fe1bd5e0bc35fefe1360fe34..783644929bb1f8ac35dafaec1f22b59c5b72d3ca 100644
--- a/content/browser/in_process_webkit/indexed_db_transaction_callbacks.cc
+++ b/content/browser/in_process_webkit/indexed_db_transaction_callbacks.cc
@@ -23,11 +23,13 @@ void IndexedDBTransactionCallbacks::onAbort() {
 }
 
 void IndexedDBTransactionCallbacks::onComplete() {
+  dispatcher_host_->TransactionComplete(transaction_id_);
   dispatcher_host_->Send(
       new IndexedDBMsg_TransactionCallbacksComplete(transaction_id_));
 }
 
 void IndexedDBTransactionCallbacks::onTimeout() {
+  NOTREACHED();

[michaeln, 2011/08/05 17:15:37]

is this truely not reached? if so the following line of code can be deleted, if
not then you can remove the NOTREACHED

... and if this is truely not reached, then there's a small pile of stuff
(webkit virtual methods and impls, ipc message defs, ipc message handlers) that
can be removed from the project. It would be good to remove that dead-code (in a
later CL). The act of doing so could have a side benefit of showing you more
about how the chrome side of this thing fits together.

[dgrogan, 2011/08/05 18:02:32]

Removed.

   dispatcher_host_->Send(
       new IndexedDBMsg_TransactionCallbacksTimeout(transaction_id_));
 }