Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1629)

Unified Diff: content/public/browser/content_browser_client.h

Issue 11234032: Webview tag creation should be using storage partitions. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Disabling session storage test for isolated apps. Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/public/browser/browser_context.h ('k') | content/public/browser/content_browser_client.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/public/browser/content_browser_client.h
diff --git a/content/public/browser/content_browser_client.h b/content/public/browser/content_browser_client.h
index 189339e8e5e038a0a5ecd4ea0588a9ec94320a13..066b146bf687e2a241bf9f6d95a7ed579f5c873d 100644
--- a/content/public/browser/content_browser_client.h
+++ b/content/public/browser/content_browser_client.h
@@ -263,23 +263,8 @@ class CONTENT_EXPORT ContentBrowserClient {
virtual net::URLRequestContext* OverrideRequestContextForURL(
const GURL& url, ResourceContext* context);
- // Allow the embedder to specify storage parititon id associated with a child
- // process.
- //
- // Child processes that have different storage partition identifiers will
- // behave as if they belong to different web browsers and not be able to
- // access each other's cookies, local storage, etc. IDs must only fit the
- // pattern [a-z0-9]* (lowercase letters or digits).
- //
- // Returns the empty string for the regular storage partition.
- virtual std::string GetStoragePartitionIdForChildProcess(
- content::BrowserContext* browser_context,
- int child_process_id);
-
- // Same as GetStoragePartitionIdForChildProcess(), but uses a site instead.
- //
- // TODO(ajwong): Replace all uses of GetStoragePartitionIdForChildProcess()
- // with this one.
+ // Allow the embedder to specify a string version of the storage partition
+ // config with a site.
virtual std::string GetStoragePartitionIdForSite(
content::BrowserContext* browser_context,
const GURL& site);
@@ -290,6 +275,21 @@ class CONTENT_EXPORT ContentBrowserClient {
virtual bool IsValidStoragePartitionId(BrowserContext* browser_context,
const std::string& partition_id);
+ // Allows the embedder to provide a storage parititon configuration for a
+ // site. A storage partition configuration includes a domain of the embedder's
+ // choice, an optional name within that domain, and whether the partition is
+ // in-memory only. The |partition_domain| is [a-z]* UTF-8 string, specifying
+ // the domain in which partitions live (similar to namespace). Within a
+ // domain, partitions can be uniquely identified by the combination of
+ // |partition_name| and |in_memory| values. When a partition is not to be
+ // persisted, the |in_memory| value must be set to true.
+ virtual void GetStoragePartitionConfigForSite(
+ content::BrowserContext* browser_context,
+ const GURL& site,
+ std::string* partition_domain,
+ std::string* partition_name,
+ bool* in_memory);
+
// Create and return a new quota permission context.
virtual QuotaPermissionContext* CreateQuotaPermissionContext();
« no previous file with comments | « content/public/browser/browser_context.h ('k') | content/public/browser/content_browser_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698