Undo in memory changes made by an update job if the commit to disk fails. Provide a more clear error message when the commit fails due to exceeding the quota.

TEST=MasterEntryFailStoreNewestCacheTest
BUG=50985
Review URL: http://codereview.chromium.org/2836075

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54667 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 4 insertions, 4 deletions

diff --git a/webkit/appcache/mock_appcache_storage.cc b/webkit/appcache/mock_appcache_storage.cc
index 7e86ce4..6bf3e8d 100644
--- a/webkit/appcache/mock_appcache_storage.cc
+++ b/webkit/appcache/mock_appcache_storage.cc
@@ -189,7 +189,7 @@ void MockAppCacheStorage::ProcessStoreGroupAndNewestCache(
   Delegate* delegate = delegate_ref->delegate;
   if (simulate_store_group_and_newest_cache_failure_) {
     if (delegate)
-      delegate->OnGroupAndNewestCacheStored(group, newest_cache, false);
+      delegate->OnGroupAndNewestCacheStored(group, newest_cache, false, false);
     return;
   }
 
@@ -206,7 +206,7 @@ void MockAppCacheStorage::ProcessStoreGroupAndNewestCache(
   }
 
   if (delegate)
-    delegate->OnGroupAndNewestCacheStored(group, newest_cache, true);
+    delegate->OnGroupAndNewestCacheStored(group, newest_cache, true, false);
 }
 
 namespace {
@@ -434,8 +434,8 @@ bool MockAppCacheStorage::ShouldGroupLoadAppearAsync(
   // The LoadGroup interface implies also loading the newest cache, so
   // if loading the newest cache should appear async, so too must the
   // loading of this group.
-  if (ShouldCacheLoadAppearAsync(group->newest_complete_cache()))
-    return true;
+  if (!ShouldCacheLoadAppearAsync(group->newest_complete_cache()))
+    return false;
 
 
   // If any of the old caches are "in use", then the group must also