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

content/public/browser/resource_context.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_RESOURCE_CONTEXT_H_
 #define CONTENT_PUBLIC_BROWSER_RESOURCE_CONTEXT_H_
 
+#include "base/basictypes.h"
 #include "base/supports_user_data.h"
 #include "content/common/content_export.h"
 
 namespace appcache {
 class AppCacheService;
 }
 
 namespace net {
 class HostResolver;
 class URLRequestContext;
 }
 
 namespace content {
 
 // ResourceContext contains the relevant context information required for
 // resource loading. It lives on the IO thread, although it is constructed on
 // the UI thread. It must be destructed on the IO thread.
 class CONTENT_EXPORT ResourceContext : public base::SupportsUserData {
  public:
   static appcache::AppCacheService* GetAppCacheService(
       ResourceContext* resource_context);
 
   ResourceContext();
   virtual ~ResourceContext();
   virtual net::HostResolver* GetHostResolver() = 0;
   virtual net::URLRequestContext* GetRequestContext() = 0;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(ResourceContext);

[jam, 2012/08/15 15:16:22]

ditto

[awong, 2012/08/17 00:52:39]

I had taken the convention to redeclare these things similar to how we redeclare
"virtual."  Is this something you'd rather I remove?

[jam, 2012/08/17 16:42:24]

I think this is a little different from putting "virtual" in derived classes. in
this case, since a class with pure virtual functions isn't copyable,
DISALLOW_COPY_AND_ASSIGN is unnecessary.

 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_RESOURCE_CONTEXT_H_