Move SessionStorageNamespace entirely into NavigationController and support StoragePartitions.

chrome/browser/chrome_content_browser_client.h

Index: chrome/browser/chrome_content_browser_client.h
diff --git a/chrome/browser/chrome_content_browser_client.h b/chrome/browser/chrome_content_browser_client.h
index 6e4cb51ab9e90b20953237ac70632be5e9908b0e..bab993b9fc250040787ece4180685831a7a8f92e 100644
--- a/chrome/browser/chrome_content_browser_client.h
+++ b/chrome/browser/chrome_content_browser_client.h
@@ -37,6 +37,12 @@ class ChromeContentBrowserClient : public content::ContentBrowserClient {
   virtual std::string GetStoragePartitionIdForChildProcess(
       content::BrowserContext* browser_context,
       int child_process_id) OVERRIDE;
+  virtual std::string GetStoragePartitionIdForSiteInstance(

[Charlie Reis, 2012/08/06 20:38:30]

Do we need the "ForX" in these two names, since child_process_id and
SiteInstance are already in the parameter list?

[awong, 2012/08/08 18:08:28]

The style guide doesn't like overloading. :-/

Also, I want one of these to go away.  But I need to get further in the
refactoring first.

+      content::BrowserContext* browser_context,
+      content::SiteInstance* instance) OVERRIDE;
+  virtual bool IsValidStoragePartitionId(
+      content::BrowserContext* browser_context,
+      const std::string& partition_id) OVERRIDE;
   virtual content::WebContentsViewDelegate* GetWebContentsViewDelegate(
       content::WebContents* web_contents) OVERRIDE;
   virtual void RenderViewHostCreated(