Move StoragePartition into content/public and remove BrowserContext::GetDOMStorageContext().

content/browser/renderer_host/render_process_host_impl.h

Index: content/browser/renderer_host/render_process_host_impl.h
diff --git a/content/browser/renderer_host/render_process_host_impl.h b/content/browser/renderer_host/render_process_host_impl.h
index 1d568addb6c475004b3ff733919ff303bc5de25d..8e36e590704a2886b33529cf9f8efd3982d9c354 100644
--- a/content/browser/renderer_host/render_process_host_impl.h
+++ b/content/browser/renderer_host/render_process_host_impl.h
@@ -26,6 +26,7 @@ class RendererMainThread;
 class RenderWidgetHelper;
 class RenderWidgetHost;
 class RenderWidgetHostImpl;
+class StoragePartitionImpl;
 
 // Implements a concrete RenderProcessHost for the browser process for talking
 // to actual renderer processes (as opposed to mocks).
@@ -45,7 +46,9 @@ class CONTENT_EXPORT RenderProcessHostImpl
     : public RenderProcessHost,
       public ChildProcessLauncher::Client {
  public:
-  RenderProcessHostImpl(BrowserContext* browser_context, bool is_guest);
+  RenderProcessHostImpl(BrowserContext* browser_context,
+                        StoragePartitionImpl* storage_partition_impl,

[Charlie Reis, 2012/08/14 18:50:12]

Why is this a StoragePartitionImpl rather than a StoragePartition?  Looks like
we only use the public API.

[awong, 2012/08/14 19:12:47]

We get covariant return which removes the need to downcast
DOMStorageContextImpl, IndexDBContextImp, and ChromeAppCacheService in the
implementation.

+                        bool is_guest);
   virtual ~RenderProcessHostImpl();
 
   // RenderProcessHost implementation (public portion).
@@ -259,6 +262,9 @@ class CONTENT_EXPORT RenderProcessHostImpl
 
   BrowserContext* browser_context_;
 
+  // Owned by |browser_context_|.
+  StoragePartitionImpl* storage_partition_impl_;
+
   // True if the process can be shut down suddenly.  If this is true, then we're
   // sure that all the RenderViews in the process can be shutdown suddenly.  If
   // it's false, then specific RenderViews might still be allowed to be shutdown