Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(204)

Unified Diff: webkit/appcache/mock_appcache_storage_unittest.cc

Issue 3529009: Fix http/tests/appcache/foreign-fallback.html (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webkit/appcache/mock_appcache_storage_unittest.cc
===================================================================
--- webkit/appcache/mock_appcache_storage_unittest.cc (revision 64524)
+++ webkit/appcache/mock_appcache_storage_unittest.cc (working copy)
@@ -44,11 +44,13 @@
}
void OnMainResponseFound(const GURL& url, const AppCacheEntry& entry,
+ const GURL& fallback_url,
const AppCacheEntry& fallback_entry,
int64 cache_id, const GURL& manifest_url,
bool was_blocked_by_policy) {
found_url_ = url;
found_entry_ = entry;
+ found_fallback_url_ = fallback_url;
found_fallback_entry_ = fallback_entry;
found_cache_id_ = cache_id;
found_manifest_url_ = manifest_url;
@@ -64,6 +66,7 @@
bool obsoleted_success_;
GURL found_url_;
AppCacheEntry found_entry_;
+ GURL found_fallback_url_;
AppCacheEntry found_fallback_entry_;
int64 found_cache_id_;
GURL found_manifest_url_;
@@ -399,6 +402,7 @@
EXPECT_EQ(kNoCacheId, delegate.found_cache_id_);
EXPECT_EQ(kNoResponseId, delegate.found_entry_.response_id());
EXPECT_EQ(kNoResponseId, delegate.found_fallback_entry_.response_id());
+ EXPECT_TRUE(delegate.found_fallback_url_.is_empty());
EXPECT_EQ(0, delegate.found_entry_.types());
EXPECT_EQ(0, delegate.found_fallback_entry_.types());
}
@@ -490,6 +494,7 @@
EXPECT_EQ(kCacheId, delegate.found_cache_id_);
EXPECT_FALSE(delegate.found_entry_.has_response_id());
EXPECT_EQ(kResponseId2, delegate.found_fallback_entry_.response_id());
+ EXPECT_EQ(kFallbackEntryUrl2, delegate.found_fallback_url_);
EXPECT_TRUE(delegate.found_fallback_entry_.IsFallback());
}
@@ -592,6 +597,7 @@
EXPECT_EQ(kNoCacheId, delegate.found_cache_id_);
EXPECT_EQ(kNoResponseId, delegate.found_entry_.response_id());
EXPECT_EQ(kNoResponseId, delegate.found_fallback_entry_.response_id());
+ EXPECT_TRUE(delegate.found_fallback_url_.is_empty());
EXPECT_EQ(0, delegate.found_entry_.types());
EXPECT_EQ(0, delegate.found_fallback_entry_.types());
@@ -605,6 +611,7 @@
EXPECT_EQ(kNoCacheId, delegate.found_cache_id_);
EXPECT_EQ(kNoResponseId, delegate.found_entry_.response_id());
EXPECT_EQ(kNoResponseId, delegate.found_fallback_entry_.response_id());
+ EXPECT_TRUE(delegate.found_fallback_url_.is_empty());
EXPECT_EQ(0, delegate.found_entry_.types());
EXPECT_EQ(0, delegate.found_fallback_entry_.types());
}

Powered by Google App Engine
This is Rietveld 408576698