Remove storage context accessors from ResourceContext.

content/public/browser/storage_partition.h

Index: content/public/browser/storage_partition.h
diff --git a/content/public/browser/storage_partition.h b/content/public/browser/storage_partition.h
index d7c13630cd7414d5676f6f1cfe30c012dfb1089b..1f5288bcebfabc59bd63f4f89f6e9c410c4a0815 100644
--- a/content/public/browser/storage_partition.h
+++ b/content/public/browser/storage_partition.h
@@ -5,6 +5,8 @@
 #ifndef CONTENT_PUBLIC_BROWSER_STORAGE_PARTITION_H_
 #define CONTENT_PUBLIC_BROWSER_STORAGE_PARTITION_H_
 
+#include <string>
+
 #include "base/basictypes.h"
 
 namespace appcache {
@@ -37,6 +39,14 @@ class DOMStorageContext;
 // the cookies, localStorage, etc., that normal web renderers have access to.
 class StoragePartition {
  public:
+  // Returns the partition id for this StoragePartition.
+  //
+  // This value should be used to compare whether two StoragePartition objects
+  // are equivalent. On the IO thread, this can be particularly useful because
+  // StoragePartition objects themselves are only meant for use on the UI
+  // thread.

[michaeln, 2012/08/30 21:48:45]

I'm thinking that we could make this class thread safe refcounted and establish
relationships needed to glue things together on the IO thread roughly as
follows...

Each RenderProcessHost and WorkerProcessHost that's bound to a particular
partition would hold a reference to this object. The RenderProcessHost's
internal RenderWidgetHelper (helper that spans the IO and UI thread) would
actually hold that ref on behalf of the RenderProcessHost.

When it comes time to spin up a new WorkerProcessHost on the IO thread on behalf
of a particular renderer, we can navigate to the StoragePartition to be used via
RenderWidgetHelper::FromProcessHostID(renderer_id)->GetStoragePartition()). It's
data members are then used to initialize various message filters and such for
that process host.

In that world, the 'id' attribute wouldn't really be needed, ptr value
comparisons would be preferred.

+  virtual const std::string& GetId() = 0;
+
   virtual quota::QuotaManager* GetQuotaManager() = 0;
   virtual appcache::AppCacheService* GetAppCacheService() = 0;
   virtual fileapi::FileSystemContext* GetFileSystemContext() = 0;