Redo the Storage Partition directory layout to support guest tags and origin based partitions.

content/browser/storage_partition_impl.cc

 // Copyright (c) 2012 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/storage_partition_impl.h"
 
 #include "content/browser/fileapi/browser_file_system_helper.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 #include "webkit/database/database_tracker.h"
 #include "webkit/quota/quota_manager.h"
 
 namespace content {
 
+namespace {
+
+// These constants are used to create the directory structure under the profile
+// where renderers with a non-default storage partition keep their persistent
+// state. This will contain a set of directories that partially mirror the
+// directory structure of BrowserContext::GetPath().
+//
+// The kStoragePartitionDirname is contains an extensions directory which is
+// further partitioned by extension id, followed by another level of directories
+// for the "default" extension storage partition and one directory for each
+// browser tag with persistent storage. Example:

[Charlie Reis, 2012/09/14 22:37:31]

nit: This still isn't quite right, since you can have multiple browser tags with
the same persistent storage partition.  (It's not per-tag, it's per-partition.)

[awong, 2012/09/14 23:01:22]

reworded.

+//
+//   {kStoragePartitionDirname}/extensions/ABCDEF/default
+//   {kStoragePartitionDirname}/extensions/ABCDEF/{hash(guest partition)}
+//
+// The code in GetPartitionPath() constructs these path names.
+const FilePath::CharType kStoragePartitionDirname[] =
+    FILE_PATH_LITERAL("Storage Partitions");
+const char kExtensionsDirname[] = "extensions";
+const char kDefaultPartitionDirname[] = "default";
+
+}  // namespace
+
+FilePath StoragePartition::GetPartitionPath(const FilePath& profile_path,

[Charlie Reis, 2012/09/14 22:37:31]

nit: // static

[awong, 2012/09/14 23:01:22]

Done.

+                                            const std::string& partition_id) {
+  if (partition_id.empty()) {
+    // The default profile just sits inside the top-level profile directory.
+    return profile_path;
+  }
+
+  // TODO(ajwong): This should check we create a valid path name.
+  CHECK(IsStringASCII(partition_id));
+  return profile_path.Append(kStoragePartitionDirname)
+      .AppendASCII(kExtensionsDirname)
+      .AppendASCII(partition_id)
+      .AppendASCII(kDefaultPartitionDirname);
+}
+
 StoragePartitionImpl::StoragePartitionImpl(
     const FilePath& partition_path,
     quota::QuotaManager* quota_manager,
     ChromeAppCacheService* appcache_service,
     fileapi::FileSystemContext* filesystem_context,
     webkit_database::DatabaseTracker* database_tracker,
     DOMStorageContextImpl* dom_storage_context,
     IndexedDBContextImpl* indexed_db_context)
     : partition_path_(partition_path),
       quota_manager_(quota_manager),
@@ skipping 15 matching lines @@
   }
 
   if (GetDOMStorageContext())
     GetDOMStorageContext()->Shutdown();
 }
 
 // TODO(ajwong): Break the direct dependency on |context|. We only
 // need 3 pieces of info from it.
 StoragePartitionImpl* StoragePartitionImpl::Create(
     BrowserContext* context,
-    const FilePath& partition_path) {
+    const std::string& partition_id,
+    const FilePath& profile_path) {
   // Ensure that these methods are called on the UI thread, except for
   // unittests where a UI thread might not have been created.
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI) ||
          !BrowserThread::IsMessageLoopValid(BrowserThread::UI));
 
+  FilePath partition_path =
+      StoragePartition::GetPartitionPath(context->GetPath(),
+                                         partition_id);
+
   // All of the clients have to be created and registered with the
   // QuotaManager prior to the QuotaManger being used. We do them
   // all together here prior to handing out a reference to anything
   // that utilizes the QuotaManager.
   scoped_refptr<quota::QuotaManager> quota_manager =
       new quota::QuotaManager(
           context->IsOffTheRecord(), partition_path,
           BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO),
           BrowserThread::GetMessageLoopProxyForThread(BrowserThread::DB),
           context->GetSpecialStoragePolicy());
@@ skipping 26 matching lines @@
 
   return new StoragePartitionImpl(partition_path,
                                   quota_manager,
                                   appcache_service,
                                   filesystem_context,
                                   database_tracker,
                                   dom_storage_context,
                                   indexed_db_context);
 }
 
+FilePath StoragePartitionImpl::GetPath() {
+  return partition_path_;
+}
+
 quota::QuotaManager* StoragePartitionImpl::GetQuotaManager() {
   return quota_manager_;
 }
 
 ChromeAppCacheService* StoragePartitionImpl::GetAppCacheService() {
   return appcache_service_;
 }
 
 fileapi::FileSystemContext* StoragePartitionImpl::GetFileSystemContext() {
   return filesystem_context_;
 }
 
 webkit_database::DatabaseTracker* StoragePartitionImpl::GetDatabaseTracker() {
   return database_tracker_;
 }
 
 DOMStorageContextImpl* StoragePartitionImpl::GetDOMStorageContext() {
   return dom_storage_context_;
 }
 
 IndexedDBContextImpl* StoragePartitionImpl::GetIndexedDBContext() {
   return indexed_db_context_;
 }
 
 }  // namespace content