* Fleshed out AppCacheHost class a fair amount, in particular the cache selection algorithm.

* Added some AppCacheHost unit tests.

* Introduced AppCacheRequestHandler class, which replaces the clunkyApp
CacheInterceptor::ExtraInfo struct. This impl is entirely skeletal
stubs for now.

TEST=appcache_unittest.cc, but really needs more
BUG=none
Review URL: http://codereview.chromium.org/192043

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

1 files changed, 13 insertions, 14 deletions

diff --git a/webkit/appcache/appcache_unittest.cc b/webkit/appcache/appcache_unittest.cc
index 951cb8b..b5c1772 100644
--- a/webkit/appcache/appcache_unittest.cc
+++ b/webkit/appcache/appcache_unittest.cc
@@ -4,33 +4,29 @@
 
 #include "testing/gtest/include/gtest/gtest.h"
 #include "webkit/appcache/appcache.h"
+#include "webkit/appcache/appcache_frontend_impl.h"
 #include "webkit/appcache/appcache_host.h"
 #include "webkit/appcache/appcache_service.h"
 
-using appcache::AppCache;
-using appcache::AppCacheEntry;
-using appcache::AppCacheHost;
-using appcache::AppCacheService;
-
-namespace {
+namespace appcache {
 
 class AppCacheTest : public testing::Test {
 };
 
-}  // namespace
-
 TEST(AppCacheTest, CleanupUnusedCache) {
   AppCacheService service;
+  AppCacheFrontendImpl frontend;
   AppCache* cache = new AppCache(&service, 111);
+  cache->set_complete(true);
 
-  AppCacheHost host1(1, NULL);
-  AppCacheHost host2(2, NULL);
+  AppCacheHost host1(1, &frontend, &service);
+  AppCacheHost host2(2, &frontend, &service);
 
-  host1.set_selected_cache(cache);
-  host2.set_selected_cache(cache);
+  host1.AssociateCache(cache);
+  host2.AssociateCache(cache);
 
-  host1.set_selected_cache(NULL);
-  host2.set_selected_cache(NULL);
+  host1.AssociateCache(NULL);
+  host2.AssociateCache(NULL);
 }
 
 TEST(AppCacheTest, AddModifyEntry) {
@@ -53,3 +49,6 @@ TEST(AppCacheTest, AddModifyEntry) {
     cache->GetEntry(kUrl1)->types());
   EXPECT_EQ(entry2.types(), cache->GetEntry(kUrl2)->types());  // unchanged
 }
+
+}  // namespace appacache
+