AppCaches which belong to hosted apps are not protected from deletion

webkit/appcache/appcache_storage.h

Index: webkit/appcache/appcache_storage.h
diff --git a/webkit/appcache/appcache_storage.h b/webkit/appcache/appcache_storage.h
index 9ddb7c3f34ba08132dea7aa8206d6ee493076d23..74e1097d722763e44e596060aad91ae509627671 100644
--- a/webkit/appcache/appcache_storage.h
+++ b/webkit/appcache/appcache_storage.h
@@ -76,6 +76,9 @@ class AppCacheStorage {
   // stored in the system. Upon completion the delegate will be called
   // with the results.
   virtual void GetAllInfo(Delegate* delegate) = 0;
+  // The same as GetAllInfo but completes synchronously (delegate will be called
+  // within the method).
+  virtual void SyncGetAllInfo(Delegate* delegate) = 0;

[michaeln, 2011/07/14 00:42:34]

Please remove all of these Sync prefixed methods from this class and its
derivatives.

[marja(google), 2011/07/14 10:31:59]

Done.

 
   // Schedules a cache to be loaded from storage. Upon load completion
   // the delegate will be called back. If the cache already resides in
@@ -91,6 +94,10 @@ class AppCacheStorage {
   // the delegate will be called back with a NULL group pointer.
   virtual void LoadOrCreateGroup(
       const GURL& manifest_url, Delegate* delegate) = 0;
+  // The same as SyncLoadOrCreateGroup but completes synchronously (delegate
+  // will be called within the method).
+  virtual void SyncLoadOrCreateGroup(
+      const GURL& manifest_url, Delegate* delegate) = 0;
 
   // Schedules response info to be loaded from storage.
   // Upon load completion the delegate will be called back. If the data
@@ -137,6 +144,10 @@ class AppCacheStorage {
   // instance is marked as obsolete and the delegate callback is called.
   virtual void MakeGroupObsolete(
       AppCacheGroup* group, Delegate* delegate) = 0;
+  // The same as MakeGroupObsolete but completes synchronously (delegate will be
+  // called within the method).
+  virtual void SyncMakeGroupObsolete(
+      AppCacheGroup* group, Delegate* delegate) = 0;
 
   // Cancels all pending callbacks for the delegate. The delegate callbacks
   // will not be invoked after, however any scheduled operations will still
@@ -313,4 +324,3 @@ class AppCacheStorage {
 }  // namespace appcache
 
 #endif  // WEBKIT_APPCACHE_APPCACHE_STORAGE_H_
-