Improve IndexedDB's quota support

content/browser/in_process_webkit/indexed_db_transaction_callbacks.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/in_process_webkit/indexed_db_transaction_callbacks.h"
 
 #include "content/browser/in_process_webkit/indexed_db_dispatcher_host.h"
 #include "content/common/indexed_db_messages.h"
 
 IndexedDBTransactionCallbacks::IndexedDBTransactionCallbacks(
     IndexedDBDispatcherHost* dispatcher_host,
     int transaction_id)
     : dispatcher_host_(dispatcher_host),
       transaction_id_(transaction_id) {
 }
 
 IndexedDBTransactionCallbacks::~IndexedDBTransactionCallbacks() {
 }
 
 void IndexedDBTransactionCallbacks::onAbort() {

[michaeln, 2011/08/03 03:40:40]

It's also 'complete' in this onAbort case, just not successfully. How does the
transaction_size_ map get cleaned up in this case and in onTimeout?

[dgrogan, 2011/08/03 21:45:47]

This is handled in patch 6.  OnTimeout is deprecated, the timeout stuff was, or
at least should have been, removed from the spec for anything asynchronous.

[michaeln, 2011/08/04 00:23:27]

Is onTimeout() called by in the current impl? If not, can you put a
NOT_REACHED() in there?

[dgrogan, 2011/08/04 23:44:55]

Done.

   dispatcher_host_->Send(
       new IndexedDBMsg_TransactionCallbacksAbort(transaction_id_));
 }
 
 void IndexedDBTransactionCallbacks::onComplete() {
+  dispatcher_host_->TransactionComplete(transaction_id_);
   dispatcher_host_->Send(
       new IndexedDBMsg_TransactionCallbacksComplete(transaction_id_));
 }
 
 void IndexedDBTransactionCallbacks::onTimeout() {
   dispatcher_host_->Send(
       new IndexedDBMsg_TransactionCallbacksTimeout(transaction_id_));
 }