Optimizes the operation of DeleteDatabase when no backing store exists.

content/browser/indexed_db/indexed_db_factory_impl.cc

 // 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.
 
 #include "content/browser/indexed_db/indexed_db_factory_impl.h"
 
 #include <utility>
 #include <vector>
 
 #include "base/logging.h"
@@ skipping 219 matching lines @@
     // database.
     it->second->DeleteDatabase(callbacks);
     return;
   }
 
   // TODO(dgrogan): Plumb data_loss back to script eventually?
   blink::WebIDBDataLoss data_loss;
   std::string data_loss_message;
   bool disk_full = false;
   leveldb::Status s;
+  IndexedDBBackingStoreMap::iterator it2 = backing_store_map_.find(origin_url);

[cmumford, 2015/04/08 15:14:35]

Just call IsBackingStoreOpen(origin_url)

+  if(it2 == backing_store_map_.end())

[cmumford, 2015/04/08 15:14:35]

spaces, and { are incorrect. Run "git cl format" to have your formatting fixed.

+  {
+      ContextDestroyed();
+      return;
+  }

[cmumford, 2015/04/08 15:14:35]

Why not have this test at the very beginning of this function?

   scoped_refptr<IndexedDBBackingStore> backing_store =
       OpenBackingStore(origin_url,
                        data_directory,
                        request_context,
                        &data_loss,
                        &data_loss_message,
                        &disk_full,
                        &s);
   if (!backing_store.get()) {
     IndexedDBDatabaseError error(blink::WebIDBDatabaseExceptionUnknownError,
@@ skipping 244 matching lines @@
   size_t count(0);
 
   OriginDBs range = GetOpenDatabasesForOrigin(origin_url);
   for (OriginDBMapIterator it = range.first; it != range.second; ++it)
     count += it->second->ConnectionCount();
 
   return count;
 }
 
 }  // namespace content