Commit b9c83fce authored by Eric Lawrence [MSFT]'s avatar Eric Lawrence [MSFT] Committed by Commit Bot

Update storage readme.md

Onion-souping landed long ago, so remove the TODO item in the doc.

Change-Id: I58a20028eeae43fd56ac37f86d3be20e5b01b315
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2547944
Auto-Submit: Eric Lawrence [MSFT] <ericlaw@microsoft.com>
Reviewed-by: default avatarMarijn Kruisselbrink <mek@chromium.org>
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#828963}
parent 2ed1b2eb
...@@ -4,11 +4,6 @@ This directory contains the renderer side implementation of the DOM Storage API. ...@@ -4,11 +4,6 @@ This directory contains the renderer side implementation of the DOM Storage API.
The browser side code for this lives in `content/browser/dom_storage/`. The browser side code for this lives in `content/browser/dom_storage/`.
*TODO(dmurph): Delete this paragraph after onion-souping is complete.
https://crbug.com/781870*
This file describes only the post-onion-souped version of the code, where
`features::kOnionSoupDOMStorage` is turned on. This is not yet the default.
## Class Responsibilities ## Class Responsibilities
### [`DOMWindowStorage`](dom_window_storage.h) ### [`DOMWindowStorage`](dom_window_storage.h)
This implements the partial `Window` interface in This implements the partial `Window` interface in
...@@ -47,7 +42,7 @@ This class is responsible for ...@@ -47,7 +42,7 @@ This class is responsible for
[`SessionStorageNamespace`](../../../public/mojom/dom_storage/session_storage_namespace.mojom) [`SessionStorageNamespace`](../../../public/mojom/dom_storage/session_storage_namespace.mojom)
mojo interfaces to create `StorageArea` mojo ptrs for the `CachedStorageArea`s mojo interfaces to create `StorageArea` mojo ptrs for the `CachedStorageArea`s
and clone namespaces for SessionStorage. and clone namespaces for SessionStorage.
* accounting for all storage used in it's cached areas, and * accounting for all storage used in its cached areas, and
* cleaning up unused caches areas on demand. * cleaning up unused caches areas on demand.
There are two versions of this class - one version is the SessionStorage There are two versions of this class - one version is the SessionStorage
...@@ -60,7 +55,7 @@ to open the `StorageAreas`, and is owned by the `StorageController`. ...@@ -60,7 +55,7 @@ to open the `StorageAreas`, and is owned by the `StorageController`.
### [`StorageController`](storage_controller.h) ### [`StorageController`](storage_controller.h)
This is a singleton that is responsible for keeping track of all This is a singleton that is responsible for keeping track of all
`StorageNamespace`s and provide access to the functionality of it's single `StorageNamespace`s and provide access to the functionality of its single
LocalStorage `StorageNamespace` that it owns. It holds weak references to all LocalStorage `StorageNamespace` that it owns. It holds weak references to all
of the SessionStorage `StorageNamespace`s so it can account for the total of the SessionStorage `StorageNamespace`s so it can account for the total
amount of memory used by all of DOMStorage. If this gets too high, it can ask amount of memory used by all of DOMStorage. If this gets too high, it can ask
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment