AppCaches which belong to hosted apps are not protected from deletion

content/browser/appcache/chrome_appcache_service.cc

 // Copyright (c) 2011 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/appcache/chrome_appcache_service.h"
 
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "content/browser/content_browser_client.h"
 #include "content/common/notification_service.h"
 #include "net/base/net_errors.h"
 #include "webkit/appcache/appcache_thread.h"
 #include "webkit/quota/quota_manager.h"
 
 static bool has_initialized_thread_ids;
 
-namespace {
-
-// Used to defer deleting of local storage until the destructor has finished.
-void DeleteLocalStateOnIOThread(FilePath cache_path) {
-  // Post the actual deletion to the DB thread to ensure it happens after the
-  // database file has been closed.
-  BrowserThread::PostTask(
-      BrowserThread::DB, FROM_HERE,
-      NewRunnableFunction<bool(*)(const FilePath&, bool), FilePath, bool>(
-          &file_util::Delete, cache_path, true));
-}
-
-}  // namespace
-
 // ----------------------------------------------------------------------------
 
 ChromeAppCacheService::ChromeAppCacheService(
     quota::QuotaManagerProxy* quota_manager_proxy)
     : AppCacheService(quota_manager_proxy),
-      resource_context_(NULL), clear_local_state_on_exit_(false) {
+      resource_context_(NULL), clear_appcache_helper_(NULL),
+      ALLOW_THIS_IN_INITIALIZER_LIST(appcache_got_info_callback_(
+          this, &ChromeAppCacheService::OnGotAppCacheInfo)),
+      ALLOW_THIS_IN_INITIALIZER_LIST(appcache_deleted_callback_(
+          this, &ChromeAppCacheService::OnAppCacheDeleted)),
+      appcaches_to_be_deleted_count_(0),
+      appcaches_cleared_event_(new base::WaitableEvent(true, false)) {
 }
 
 void ChromeAppCacheService::InitializeOnIOThread(
     const FilePath& cache_path,
     const content::ResourceContext* resource_context,
-    scoped_refptr<quota::SpecialStoragePolicy> special_storage_policy,
-    bool clear_local_state_on_exit) {
+    scoped_refptr<quota::SpecialStoragePolicy> special_storage_policy) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
 
   if (!has_initialized_thread_ids) {
     has_initialized_thread_ids = true;
     appcache::AppCacheThread::Init(BrowserThread::DB, BrowserThread::IO);
   }
 
   cache_path_ = cache_path;
   resource_context_ = resource_context;
   registrar_.Add(
       this, NotificationType::PURGE_MEMORY, NotificationService::AllSources());
-  SetClearLocalStateOnExit(clear_local_state_on_exit);
 
   // Init our base class.
   Initialize(cache_path_,
              BrowserThread::GetMessageLoopProxyForThread(BrowserThread::CACHE));
   set_appcache_policy(this);
   set_special_storage_policy(special_storage_policy);
 }
 
 ChromeAppCacheService::~ChromeAppCacheService() {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
-
-  if (clear_local_state_on_exit_ && !cache_path_.empty()) {
-    BrowserThread::PostTask(
-        BrowserThread::IO, FROM_HERE,
-        NewRunnableFunction(DeleteLocalStateOnIOThread, cache_path_));
-  }
 }
 
-void ChromeAppCacheService::SetClearLocalStateOnExit(bool clear_local_state) {
-  // TODO(michaeln): How is 'protected' status granted to apps in this case?
+base::WaitableEvent* ChromeAppCacheService::ClearAppCache(
+    net::CompletionCallback* callback)
+{
   if (!BrowserThread::CurrentlyOn(BrowserThread::IO)) {
     BrowserThread::PostTask(
-        BrowserThread::IO, FROM_HERE,
-        NewRunnableMethod(this,
-                          &ChromeAppCacheService::SetClearLocalStateOnExit,
-                          clear_local_state));
-    return;
+      BrowserThread::IO, FROM_HERE,
+      NewRunnableMethod(this,
+                        &ChromeAppCacheService::ClearAppCache,
+                        callback));
+    return appcaches_cleared_event_.get();
   }
-  clear_local_state_on_exit_ = clear_local_state;
+
+  // FIXME: should we handle multiple simultaneous deletions better?
+  if (clear_appcache_helper_)
+    return appcaches_cleared_event_.get();
+
+  appcaches_cleared_event_->Reset();
+  // ClearAppCacheHelper keeps this ChromeAppCacheService alive until the
+  // deletion is done, even if other objects drop their references.
+  clear_appcache_helper_ = new ClearAppcacheHelper(this, callback);

[jochen (gone - plz use gerrit), 2011/07/06 09:41:45]

you could just this->Ref(); and Deref() later, no need to have an extra class

[marja(google), 2011/07/06 14:34:09]

Done.

[marja(google), 2011/07/13 11:06:59]

Done.

+
+  appcache_info_ = new appcache::AppCacheInfoCollection;
+  GetAllAppCacheInfo(
+    appcache_info_, &appcache_got_info_callback_);
+  // continues in OnGotAppCacheInfo
+
+  return appcaches_cleared_event_.get();
+}
+
+void ChromeAppCacheService::OnGotAppCacheInfo(int rv) {
+  using appcache::AppCacheInfoVector;
+  typedef std::map<GURL, AppCacheInfoVector> InfoByOrigin;
+
+  appcaches_to_be_deleted_count_ = 0;
+  for (InfoByOrigin::const_iterator origin =
+           appcache_info_->infos_by_origin.begin();
+       origin != appcache_info_->infos_by_origin.end(); ++origin) {
+
+    if (special_storage_policy_->IsStorageProtected(origin->first))
+      continue;
+
+    ++appcaches_to_be_deleted_count_;

[michaeln, 2011/07/02 00:29:09]

maybe origins_to_be_deleted_count_

[marja(google), 2011/07/06 14:34:09]

Done.

[marja(google), 2011/07/13 11:06:59]

I had changed this, but changed it back because of the DeleteAppCachesForOrigin
-> DeleteAppCacheGroup change. Now it's appcaches_to_be_deleted_count_ like it
was before, but now we're counting appcaches, not origins.

+    DeleteAppCachesForOrigin(origin->first, &appcache_deleted_callback_);

[michaeln, 2011/07/02 00:29:09]

maybe origin_deleted_callback_

[marja(google), 2011/07/06 14:34:09]

Done.

[marja(google), 2011/07/13 11:06:59]

Same here.

+  }
+
+  if (appcaches_to_be_deleted_count_ == 0)
+    clear_appcache_helper_->OnClearedAppCache();
+  // else continues in OnAppCacheDeleted
+}
+
+void ChromeAppCacheService::OnAppCacheDeleted(int rv) {
+  --appcaches_to_be_deleted_count_;
+  if (appcaches_to_be_deleted_count_ == 0)
+    clear_appcache_helper_->OnClearedAppCache();
 }
 
 bool ChromeAppCacheService::CanLoadAppCache(const GURL& manifest_url) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   // We don't prompt for read access.
   return content::GetContentClient()->browser()->AllowAppCache(
       manifest_url, *resource_context_);
 }
 
 int ChromeAppCacheService::CanCreateAppCache(
     const GURL& manifest_url, net::CompletionCallback* callback) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   return content::GetContentClient()->browser()->AllowAppCache(
       manifest_url, *resource_context_) ? net::OK : net::ERR_ACCESS_DENIED;
 }
 
 void ChromeAppCacheService::Observe(NotificationType type,
                                     const NotificationSource& source,
                                     const NotificationDetails& details) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   DCHECK(type == NotificationType::PURGE_MEMORY);
   PurgeMemory();
 }
 
 // ----------------------------------------------------------------------------
 
+ChromeAppCacheService::ClearAppcacheHelper::ClearAppcacheHelper(
+  ChromeAppCacheService* parent, net::CompletionCallback* callback)
+    : parent_(parent), callback_(callback)
+{

[michaeln, 2011/07/02 00:29:09]

style: bracket belongs at the end of the previous line

[marja(google), 2011/07/06 14:34:09]

Done.

+}
+
+ChromeAppCacheService::ClearAppcacheHelper::~ClearAppcacheHelper()
+{

[michaeln, 2011/07/02 00:29:09]

style: ditto

[marja(google), 2011/07/06 14:34:09]

Done.

+}
+
+void ChromeAppCacheService::ClearAppcacheHelper::OnClearedAppCache()

[michaeln, 2011/07/02 00:29:09]

This class seems heavier weight than it needs to be for what it does... consider
just CACS->AddRef() and CACS->Release() at the new/delete callsites of this
object instead of having a new class/object.

[marja(google), 2011/07/06 14:34:09]

Done.

+{
+  parent_->appcaches_cleared_event_->Signal();
+  parent_->clear_appcache_helper_ = NULL;
+  if (callback_) {
+    callback_->Run(0);
+  }
+  MessageLoop::current()->DeleteSoon(FROM_HERE, this);
+}
+
+
+// ----------------------------------------------------------------------------
+
 static BrowserThread::ID ToBrowserThreadID(int id) {
   DCHECK(has_initialized_thread_ids);
   DCHECK(id == BrowserThread::DB || id == BrowserThread::IO);
   return static_cast<BrowserThread::ID>(id);
 }
 
 namespace appcache {
 
 // An impl of AppCacheThread we need to provide to the appcache lib.
 
 bool AppCacheThread::PostTask(
     int id,
     const tracked_objects::Location& from_here,
     Task* task) {
   return BrowserThread::PostTask(ToBrowserThreadID(id), from_here, task);
 }
 
 bool AppCacheThread::CurrentlyOn(int id) {
   return BrowserThread::CurrentlyOn(ToBrowserThreadID(id));
 }
 
 }  // namespace appcache