Moved deleting the indexed db context to the WebKitContext destructor.

chrome/browser/in_process_webkit/indexed_db_context.h

Index: chrome/browser/in_process_webkit/indexed_db_context.h
diff --git a/chrome/browser/in_process_webkit/indexed_db_context.h b/chrome/browser/in_process_webkit/indexed_db_context.h
index 95f6536e9c15feddadefbb48a6f6550d21cd9e00..b81f1449b92d866f47fb8ec21451f1f6ac52ff31 100644
--- a/chrome/browser/in_process_webkit/indexed_db_context.h
+++ b/chrome/browser/in_process_webkit/indexed_db_context.h
@@ -35,9 +35,13 @@ class IndexedDBContext {
 
   // Deletes all idb files except for those on the url scheme
   // |url_scheme_to_be_skipped|.
-  static void ClearLocalState(const FilePath& profile_path,
+  static void ClearLocalState(const FilePath& indexeddb_path,

[pastarmovj, 2010/11/27 20:45:15]

I hope you will agree with this change. It allows for much cleaner code around
calling this function and moreover it has nothing to search in another places of
the profile anyways. 

                               const char* url_scheme_to_be_skipped);
 
+  void SetClearLocalStateOnExit(bool clear_local_state) {
+    clear_local_state_on_exit_ = clear_local_state;
+  }

[pastarmovj, 2010/11/27 20:45:15]

Do I got that correctly that really simple setters and getters can be
implemented inline even if they don't fit a single line?

+
   // Deletes a single indexed db file.
   void DeleteIndexedDBFile(const FilePath& file_path);
 
@@ -47,8 +51,11 @@ class IndexedDBContext {
  private:
   scoped_ptr<WebKit::WebIDBFactory> idb_factory_;
 
-  // We're owned by this WebKit context.
-  WebKitContext* webkit_context_;
+  // Path where the indexed db data is stored
+  FilePath data_path_;

[pastarmovj, 2010/11/27 20:45:15]

I replaced the webkit context pointer with the piece of data that we really
needed out of it. It is used in the destructor of this object and we can't rely
on having the parent object around that time so we have to cache this and
therefore the other pointer becomes obsolete.

+
+  // True if the destructor should delete its files.
+  bool clear_local_state_on_exit_;
 
   DISALLOW_COPY_AND_ASSIGN(IndexedDBContext);
 };