Convert WebIDBDatabaseImpl to IndexedDBConnection

content/browser/indexed_db/indexed_db_connection.h

+// Copyright (c) 2013 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.
+
+#ifndef CONTENT_BROWSER_INDEXED_DB_INDEXED_DB_CONNECTION_H_
+#define CONTENT_BROWSER_INDEXED_DB_INDEXED_DB_CONNECTION_H_
+
+#include "base/memory/ref_counted.h"
+#include "content/browser/indexed_db/indexed_db_database.h"
+#include "content/browser/indexed_db/indexed_db_database_callbacks.h"
+
+namespace content {
+class IndexedDBCallbacks;
+class IndexedDBDatabaseError;
+
+class CONTENT_EXPORT IndexedDBConnection {
+ public:
+  IndexedDBConnection(scoped_refptr<IndexedDBDatabase> db,
+                      scoped_refptr<IndexedDBDatabaseCallbacks> callbacks);
+  virtual ~IndexedDBConnection();
+
+  virtual void ForceClose();
+  virtual void Close();
+
+  IndexedDBDatabase* database() { return database_.get(); }
+  IndexedDBDatabaseCallbacks* callbacks() { return callbacks_; }

[dgrogan, 2013/07/02 22:01:58]

These should be consistent in whether or not they call get().

[jsbell, 2013/07/02 22:19:24]

Done. Also, see follow-up patch https://codereview.chromium.org/18147009/

+
+ private:
+  // Only NULL in tests.
+  scoped_refptr<IndexedDBDatabase> database_;
+
+  // The database_callbacks_ member is cleared when the connection

[dgrogan, 2013/07/02 22:01:58]

s/database_//

[jsbell, 2013/07/02 22:19:24]

Done.

+  // is closed. (May be NULL in tests.)
+  scoped_refptr<IndexedDBDatabaseCallbacks> callbacks_;
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_INDEXED_DB_INDEXED_DB_CONNECTION_H_