| Index: content/browser/renderer_host/render_message_filter.cc
|
| ===================================================================
|
| --- content/browser/renderer_host/render_message_filter.cc (revision 127221)
|
| +++ content/browser/renderer_host/render_message_filter.cc (working copy)
|
| @@ -411,9 +411,21 @@
|
| return;
|
| }
|
|
|
| +#ifdef ENABLE_NEW_DOM_STORAGE_BACKEND
|
| + // TODO(michaeln): Fix this.
|
| + // This is a bug in the existing impl, session storage is effectively
|
| + // leaked when created thru this code path (window.open()) since there
|
| + // is no balancing DeleteSessionStorage() for this Clone() call anywhere
|
| + // in the codebase. I'm replicating the bug for now.
|
| *cloned_session_storage_namespace_id =
|
| + dom_storage_context_->LeakyCloneSessionStorage(
|
| + params.session_storage_namespace_id);
|
| +#else
|
| + *cloned_session_storage_namespace_id =
|
| dom_storage_context_->CloneSessionStorage(
|
| params.session_storage_namespace_id);
|
| +#endif
|
| +
|
| render_widget_helper_->CreateNewWindow(params,
|
| peer_handle(),
|
| route_id,
|
|
|