Appcache::OnCorruptionDetected handling

webkit/browser/appcache/appcache_storage_impl.h

Index: webkit/browser/appcache/appcache_storage_impl.h
diff --git a/webkit/browser/appcache/appcache_storage_impl.h b/webkit/browser/appcache/appcache_storage_impl.h
index 163001d52830336168b9b8a467da00cc3951f75d..46dcce5c8c8e067ec960adac6d8227ad9893848a 100644
--- a/webkit/browser/appcache/appcache_storage_impl.h
+++ b/webkit/browser/appcache/appcache_storage_impl.h
@@ -67,8 +67,6 @@ class AppCacheStorageImpl : public AppCacheStorage {
   virtual void PurgeMemory() OVERRIDE;
 
  private:
-  friend class AppCacheStorageImplTest;
-
   // The AppCacheStorageImpl class methods and datamembers may only be
   // accessed on the IO thread. This class manufactures seperate DatabaseTasks
   // which access the DB on a seperate background thread.
@@ -114,7 +112,9 @@ class AppCacheStorageImpl : public AppCacheStorage {
 
   void OnDeletedOneResponse(int rv);
   void OnDiskCacheInitialized(int rv);
-  void CallReinitialize();
+  void DeleteAndStartOver();
+  void DeleteAndStartOverPart2();

[kinuko, 2014/01/29 07:40:30]

(totally optional) Part2 sounds nice, but it might be slightly clearer to say
what each does like...

PostDeleteAndStartOver()
DeleteAndStartOver()

+  void CallScheduleReinitialize();
 
   // Sometimes we can respond without having to query the database.
   bool FindResponseForMainRequestInGroup(
@@ -172,6 +172,7 @@ class AppCacheStorageImpl : public AppCacheStorage {
   std::deque<base::Closure> pending_simple_tasks_;
   base::WeakPtrFactory<AppCacheStorageImpl> weak_factory_;
 
+  friend class AppCacheStorageImplTest;
   friend class content::ChromeAppCacheServiceTest;
 };