Moved deleting the indexed db context to the WebKitContext destructor.

chrome/browser/in_process_webkit/indexed_db_context.h

 // Copyright (c) 2010 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 CHROME_BROWSER_IN_PROCESS_WEBKIT_INDEXED_DB_CONTEXT_H_
 #define CHROME_BROWSER_IN_PROCESS_WEBKIT_INDEXED_DB_CONTEXT_H_
 #pragma once
 
 #include "base/basictypes.h"
 #include "base/file_path.h"
@@ skipping 17 matching lines @@
   static const FilePath::CharType kIndexedDBDirectory[];
 
   // The indexed db file extension.
   static const FilePath::CharType kIndexedDBExtension[];
 
   // Get the file name of the indexed db file for the given origin.
   FilePath GetIndexedDBFilePath(const string16& origin_id) const;
 
   // 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);
 
   // Deletes all indexed db files for the given origin.
   void DeleteIndexedDBForOrigin(const string16& origin_id);
 
  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);
 };
 
 #endif  // CHROME_BROWSER_IN_PROCESS_WEBKIT_INDEXED_DB_CONTEXT_H_