Clear up session only storage on localstorage shutdown

content/browser/dom_storage/dom_storage_context_wrapper.cc

 // Copyright 2013 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.
 
 #include "content/browser/dom_storage/dom_storage_context_wrapper.h"
 
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 110 matching lines @@
                         : data_path.AppendASCII(kSessionStorageDirectory),
       special_storage_policy,
       new DOMStorageWorkerPoolTaskRunner(std::move(primary_sequence),
                                          std::move(commit_sequence)));
 
   if (base::CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kMojoLocalStorage)) {
     base::FilePath storage_dir;
     if (!profile_path.empty())
       storage_dir = local_partition_path.AppendASCII(kLocalStorageDirectory);
-    // TODO(michaeln): Enable writing to disk when db is versioned,
-    // for now using an empty subdirectory to use an in-memory db.
-    // subdirectory_(subdirectory),
-    mojo_state_.reset(new LocalStorageContextMojo(
+    // TODO(mek): Use a SequencedTaskRunner once mojo supports that.
+    mojo_task_runner_ =
+        BrowserThread::GetTaskRunnerForThread(BrowserThread::IO);
+    mojo_state_ = new LocalStorageContextMojo(
         connector, context_->task_runner(),
         data_path.empty() ? data_path
                           : data_path.AppendASCII(kLocalStorageDirectory),
-        storage_dir));
+        storage_dir, special_storage_policy);
   }
 
   if (base::FeatureList::IsEnabled(features::kMemoryCoordinator)) {
     base::MemoryCoordinatorClientRegistry::GetInstance()->Register(this);
   } else {
     memory_pressure_listener_.reset(new base::MemoryPressureListener(
         base::Bind(&DOMStorageContextWrapper::OnMemoryPressure, this)));
   }
 }
 
 DOMStorageContextWrapper::~DOMStorageContextWrapper() {}
 
 void DOMStorageContextWrapper::GetLocalStorageUsage(
     const GetLocalStorageUsageCallback& callback) {
   DCHECK(context_.get());
   if (mojo_state_) {
-    mojo_state_->GetStorageUsage(base::BindOnce(
-        &DOMStorageContextWrapper::GotMojoLocalStorageUsage, this, callback));
+    mojo_task_runner_->PostTask(
+        FROM_HERE,
+        base::BindOnce(
+            &LocalStorageContextMojo::GetStorageUsage, mojo_state_,
+            base::BindOnce(
+                &DOMStorageContextWrapper::GotMojoLocalStorageUsage, this,
+                callback,
+                base::RetainedRef(base::ThreadTaskRunnerHandle::Get()))));
     return;
   }
   context_->task_runner()->PostShutdownBlockingTask(
       FROM_HERE, DOMStorageTaskRunner::PRIMARY_SEQUENCE,
       base::Bind(&GetLocalStorageUsageHelper,
                  std::vector<LocalStorageUsageInfo>(),
                  base::RetainedRef(base::ThreadTaskRunnerHandle::Get()),
                  base::RetainedRef(context_), callback));
 }
 
 void DOMStorageContextWrapper::GetSessionStorageUsage(
     const GetSessionStorageUsageCallback& callback) {
   DCHECK(context_.get());
   context_->task_runner()->PostShutdownBlockingTask(
       FROM_HERE, DOMStorageTaskRunner::PRIMARY_SEQUENCE,
       base::Bind(&GetSessionStorageUsageHelper,
                  base::RetainedRef(base::ThreadTaskRunnerHandle::Get()),
                  base::RetainedRef(context_), callback));
 }
 
 void DOMStorageContextWrapper::DeleteLocalStorageForPhysicalOrigin(
     const GURL& origin) {
   DCHECK(context_.get());
   context_->task_runner()->PostShutdownBlockingTask(
       FROM_HERE, DOMStorageTaskRunner::PRIMARY_SEQUENCE,
       base::Bind(&DOMStorageContextImpl::DeleteLocalStorageForPhysicalOrigin,
                  context_, origin));
   if (mojo_state_)
-    mojo_state_->DeleteStorageForPhysicalOrigin(url::Origin(origin));
+    mojo_task_runner_->PostTask(
+        FROM_HERE,
+        base::BindOnce(&LocalStorageContextMojo::DeleteStorageForPhysicalOrigin,
+                       mojo_state_, url::Origin(origin)));
 }
 
 void DOMStorageContextWrapper::DeleteLocalStorage(const GURL& origin) {
   DCHECK(context_.get());
   context_->task_runner()->PostShutdownBlockingTask(
       FROM_HERE,
       DOMStorageTaskRunner::PRIMARY_SEQUENCE,
       base::Bind(&DOMStorageContextImpl::DeleteLocalStorage, context_, origin));
   if (mojo_state_)
-    mojo_state_->DeleteStorage(url::Origin(origin));
+    mojo_task_runner_->PostTask(
+        FROM_HERE, base::BindOnce(&LocalStorageContextMojo::DeleteStorage,
+                                  mojo_state_, url::Origin(origin)));
 }
 
 void DOMStorageContextWrapper::DeleteSessionStorage(
     const SessionStorageUsageInfo& usage_info) {
   DCHECK(context_.get());
   context_->task_runner()->PostShutdownBlockingTask(
       FROM_HERE,
       DOMStorageTaskRunner::PRIMARY_SEQUENCE,
       base::Bind(&DOMStorageContextImpl::DeleteSessionStorage,
                  context_, usage_info));
@@ skipping 19 matching lines @@
       base::Bind(&DOMStorageContextImpl::StartScavengingUnusedSessionStorage,
                  context_));
 }
 
 void DOMStorageContextWrapper::SetForceKeepSessionState() {
   DCHECK(context_.get());
   context_->task_runner()->PostShutdownBlockingTask(
       FROM_HERE,
       DOMStorageTaskRunner::PRIMARY_SEQUENCE,
       base::Bind(&DOMStorageContextImpl::SetForceKeepSessionState, context_));
+  if (mojo_state_)
+    mojo_task_runner_->PostTask(
+        FROM_HERE,
+        base::BindOnce(&LocalStorageContextMojo::SetForceKeepSessionState,
+                       mojo_state_));
 }
 
 void DOMStorageContextWrapper::Shutdown() {
   DCHECK(context_.get());
-  mojo_state_.reset();
+  if (mojo_state_) {
+    mojo_task_runner_->PostTask(
+        FROM_HERE,
+        base::BindOnce(&LocalStorageContextMojo::Shutdown, mojo_state_));
+  }
   memory_pressure_listener_.reset();
   context_->task_runner()->PostShutdownBlockingTask(
       FROM_HERE,
       DOMStorageTaskRunner::PRIMARY_SEQUENCE,
       base::Bind(&DOMStorageContextImpl::Shutdown, context_));
   if (base::FeatureList::IsEnabled(features::kMemoryCoordinator)) {
     base::MemoryCoordinatorClientRegistry::GetInstance()->Unregister(this);
   }
 
 }
 
 void DOMStorageContextWrapper::Flush() {
   DCHECK(context_.get());
+
   context_->task_runner()->PostShutdownBlockingTask(
       FROM_HERE, DOMStorageTaskRunner::PRIMARY_SEQUENCE,
       base::Bind(&DOMStorageContextImpl::Flush, context_));
   if (mojo_state_)
-    mojo_state_->Flush();
+    mojo_task_runner_->PostTask(
+        FROM_HERE,
+        base::BindOnce(&LocalStorageContextMojo::Flush, mojo_state_));
 }
 
 void DOMStorageContextWrapper::OpenLocalStorage(
     const url::Origin& origin,
     mojom::LevelDBWrapperRequest request) {
   if (!mojo_state_)
     return;
-  mojo_state_->OpenLocalStorage(origin, std::move(request));
+  mojo_task_runner_->PostTask(
+      FROM_HERE, base::BindOnce(&LocalStorageContextMojo::OpenLocalStorage,
+                                mojo_state_, origin, std::move(request)));
 }
 
 void DOMStorageContextWrapper::OnMemoryPressure(
     base::MemoryPressureListener::MemoryPressureLevel memory_pressure_level) {
   DOMStorageContextImpl::PurgeOption purge_option =
       DOMStorageContextImpl::PURGE_UNOPENED;
   if (memory_pressure_level ==
       base::MemoryPressureListener::MEMORY_PRESSURE_LEVEL_CRITICAL) {
     purge_option = DOMStorageContextImpl::PURGE_AGGRESSIVE;
   }
   PurgeMemory(purge_option);
 }
 
 void DOMStorageContextWrapper::OnPurgeMemory() {
   PurgeMemory(DOMStorageContextImpl::PURGE_AGGRESSIVE);
 }
 
 void DOMStorageContextWrapper::PurgeMemory(DOMStorageContextImpl::PurgeOption
     purge_option) {
   context_->task_runner()->PostTask(
       FROM_HERE,
       base::Bind(&DOMStorageContextImpl::PurgeMemory, context_, purge_option));
   if (mojo_state_ && purge_option == DOMStorageContextImpl::PURGE_AGGRESSIVE)
-    mojo_state_->PurgeMemory();
+    mojo_task_runner_->PostTask(
+        FROM_HERE,
+        base::BindOnce(&LocalStorageContextMojo::PurgeMemory, mojo_state_));
 }
 
 void DOMStorageContextWrapper::GotMojoLocalStorageUsage(
     GetLocalStorageUsageCallback callback,
+    base::SingleThreadTaskRunner* reply_task_runner,
     std::vector<LocalStorageUsageInfo> usage) {
   context_->task_runner()->PostShutdownBlockingTask(
       FROM_HERE, DOMStorageTaskRunner::PRIMARY_SEQUENCE,
       base::Bind(&GetLocalStorageUsageHelper, base::Passed(&usage),
-                 base::RetainedRef(base::ThreadTaskRunnerHandle::Get()),
+                 base::RetainedRef(reply_task_runner),
                  base::RetainedRef(context_), callback));
 }
 
 }  // namespace content