Tell IDB frontend about data loss

content/browser/indexed_db/indexed_db_database_unittest.cc

Index: content/browser/indexed_db/indexed_db_database_unittest.cc
diff --git a/content/browser/indexed_db/indexed_db_database_unittest.cc b/content/browser/indexed_db/indexed_db_database_unittest.cc
index 251b6b0f5d4e3b179f044abd07bc66b5126aa703..3af1b0bd2e5c815343248aa96980ec795449fccf 100644
--- a/content/browser/indexed_db/indexed_db_database_unittest.cc
+++ b/content/browser/indexed_db/indexed_db_database_unittest.cc
@@ -116,7 +116,8 @@ TEST(IndexedDBDatabaseTest, ConnectionLifecycle) {
   db->OpenConnection(request1,
                      connection1,
                      transaction_id1,
-                     IndexedDBDatabaseMetadata::DEFAULT_INT_VERSION);
+                     IndexedDBDatabaseMetadata::DEFAULT_INT_VERSION,
+                     false);

[jsbell, 2013/06/14 20:42:58]

Give the argument a default value instead of passing explicit false?

[dgrogan, 2013/06/14 22:06:43]

jar@ recently reminded me that default arguments are against the style guide,
though now that I read it, it recommends simply using an overload instead. Seems
reasonable.

Simulated default argument via method overload it is.

 
   EXPECT_FALSE(backing_store->HasOneRef());  // db, connection count > 0
 
@@ -127,7 +128,8 @@ TEST(IndexedDBDatabaseTest, ConnectionLifecycle) {
   db->OpenConnection(request2,
                      connection2,
                      transaction_id2,
-                     IndexedDBDatabaseMetadata::DEFAULT_INT_VERSION);
+                     IndexedDBDatabaseMetadata::DEFAULT_INT_VERSION,
+                     false);

[jsbell, 2013/06/14 20:42:58]

Ditto?

[dgrogan, 2013/06/14 22:06:43]

Done.

 
   EXPECT_FALSE(backing_store->HasOneRef());  // local and connection
 
@@ -185,7 +187,8 @@ TEST(IndexedDBDatabaseTest, ForcedClose) {
   backend->OpenConnection(request,
                           connection,
                           upgrade_transaction_id,
-                          IndexedDBDatabaseMetadata::DEFAULT_INT_VERSION);
+                          IndexedDBDatabaseMetadata::DEFAULT_INT_VERSION,
+                          false);

[jsbell, 2013/06/14 20:42:58]

Ditto?

[dgrogan, 2013/06/14 22:06:43]

Done.

 
   const int64 transaction_id = 123;
   const std::vector<int64> scope;