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

Unified Diff: blimp/common/blob_cache/in_memory_blob_cache_unittest.cc

Issue 2056993003: Add Mojo IPC for seeding new Renderer with Browser's cached blob state. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@blobchannel-master
Patch Set: Mojo mapz Created 4 years, 6 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: blimp/common/blob_cache/in_memory_blob_cache_unittest.cc
diff --git a/blimp/common/blob_cache/in_memory_blob_cache_unittest.cc b/blimp/common/blob_cache/in_memory_blob_cache_unittest.cc
index 9cc24d71d13e4b88a0422677d63b54040f8c749a..431f8eb05cd4a8bf33521cdb9d84fa274c8e2211 100644
--- a/blimp/common/blob_cache/in_memory_blob_cache_unittest.cc
+++ b/blimp/common/blob_cache/in_memory_blob_cache_unittest.cc
@@ -44,6 +44,10 @@ TEST_F(InMemoryBlobCacheTest, SimplePutContainsAndGetOperations) {
BlobDataPtr out = cache_.Get(kFoo);
EXPECT_EQ(blob_data, out);
+
+ auto cache_state = cache_.GetCachedBlobIds();
+ EXPECT_EQ(1u, cache_state.size());
+ EXPECT_EQ(kFoo, cache_state[0]);
}
TEST_F(InMemoryBlobCacheTest, TestDuplicatePut) {
@@ -59,6 +63,10 @@ TEST_F(InMemoryBlobCacheTest, TestDuplicatePut) {
cache_.Put(kFoo, duplicate);
BlobDataPtr out2 = cache_.Get(kFoo);
EXPECT_EQ(first, out2);
+
+ auto cache_state = cache_.GetCachedBlobIds();
+ EXPECT_EQ(1u, cache_state.size());
nyquist 2016/07/15 22:13:46 Nit: It feels a little bit weird of having two tes
Kevin M 2016/07/18 16:58:16 Added multiple outputs to SimplePutContainsAndGetO
+ EXPECT_EQ(kFoo, cache_state[0]);
}
} // namespace

Powered by Google App Engine
This is Rietveld 408576698