Webview tag creation should be using storage partitions.

content/browser/browser_context.cc

Index: content/browser/browser_context.cc
diff --git a/content/browser/browser_context.cc b/content/browser/browser_context.cc
index cbfd2ca7ddfe1fb7e45b733d0950c2775ffb2ef0..fb13698a60236bb7cea7506203d6d7cce5607b22 100644
--- a/content/browser/browser_context.cc
+++ b/content/browser/browser_context.cc
@@ -37,9 +37,11 @@ namespace {
 const char* kDownloadManagerKeyName = "download_manager";
 const char* kStorageParitionMapKeyName = "content_storage_partition_map";
 
-StoragePartition* GetStoragePartitionByPartitionId(
+StoragePartition* GetStoragePartitionByPartitionDescription(

[awong, 2012/11/02 21:56:13]

Name is too long.  This is just a helper so it's almost better to name it
GetStoragePartitionHelper().

At least it's easier to read and you don't gain much for saying it's by
PartitionDescription.

[nasko, 2012/11/03 00:36:24]

Done.

     BrowserContext* browser_context,
-    const std::string& partition_id) {
+    const std::string& partition_domain,
+    const std::string& partition_name,
+    const bool& in_memory) {
   StoragePartitionImplMap* partition_map =
       static_cast<StoragePartitionImplMap*>(
           browser_context->GetUserData(kStorageParitionMapKeyName));
@@ -48,12 +50,11 @@ StoragePartition* GetStoragePartitionByPartitionId(
     browser_context->SetUserData(kStorageParitionMapKeyName, partition_map);
   }
 
-  return partition_map->Get(partition_id);
+  return partition_map->Get(partition_domain, partition_name, in_memory);
 }
 
 // Run |callback| on each DOMStorageContextImpl in |browser_context|.
-void PurgeDOMStorageContextInPartition(const std::string& id,
-                                       StoragePartition* storage_partition) {
+void PurgeDOMStorageContextInPartition(StoragePartition* storage_partition) {
   static_cast<StoragePartitionImpl*>(storage_partition)->
       GetDOMStorageContext()->PurgeMemory();
 }
@@ -104,26 +105,34 @@ DownloadManager* BrowserContext::GetDownloadManager(
 StoragePartition* BrowserContext::GetStoragePartition(
     BrowserContext* browser_context,
     SiteInstance* site_instance) {
-  std::string partition_id;  // Default to "" for NULL |site_instance|.
+  std::string partition_domain;
+  std::string partition_name;
+  bool in_memory = false;
 
   // TODO(ajwong): After GetDefaultStoragePartition() is removed, get rid of
   // this conditional and require that |site_instance| is non-NULL.
   if (site_instance) {
-    partition_id = GetContentClient()->browser()->
-        GetStoragePartitionIdForSite(browser_context,
-                                     site_instance->GetSiteURL());
+    GetContentClient()->browser()->GetStoragePartitionDescriptionForSite(
+        browser_context, site_instance->GetSiteURL(),
+        &partition_domain, &partition_name, &in_memory);
   }
 
-  return GetStoragePartitionByPartitionId(browser_context, partition_id);
+  return GetStoragePartitionByPartitionDescription(
+      browser_context, partition_domain, partition_name, in_memory);
 }
 
 StoragePartition* BrowserContext::GetStoragePartitionForSite(
     BrowserContext* browser_context,
     const GURL& site) {
-  std::string partition_id = GetContentClient()->browser()->
-      GetStoragePartitionIdForSite(browser_context, site);
+  std::string partition_domain;
+  std::string partition_name;
+  bool in_memory;
 
-  return GetStoragePartitionByPartitionId(browser_context, partition_id);
+  GetContentClient()->browser()->GetStoragePartitionDescriptionForSite(
+      browser_context, site, &partition_domain, &partition_name, &in_memory);
+
+  return GetStoragePartitionByPartitionDescription(
+      browser_context, partition_domain, partition_name, in_memory);
 }
 
 void BrowserContext::ForEachStoragePartition(