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

Unified Diff: blimp/common/blob_cache/blob_cache.h

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: use a better upstream branch for the patch 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/blob_cache.h
diff --git a/blimp/common/blob_cache/blob_cache.h b/blimp/common/blob_cache/blob_cache.h
index 1f8a2932723640f2346321797ca98e29cbcb9f5d..b1c7118f102510495fc5a25257c989de31bb07a1 100644
--- a/blimp/common/blob_cache/blob_cache.h
+++ b/blimp/common/blob_cache/blob_cache.h
@@ -25,6 +25,9 @@ class BLIMP_COMMON_EXPORT BlobCache {
public:
virtual ~BlobCache() {}
+ // Gets the keys of items stored by the cache.
+ virtual std::vector<BlobId> GetCacheState() const = 0;
Wez 2016/06/21 00:33:33 nit: GetCachedBlobIds()?
Wez 2016/06/21 00:33:33 In general this seems like a potentially large str
Kevin M 2016/06/21 21:23:51 Done.
Kevin M 2016/06/21 21:23:51 Agreed, but in general I think if this saves us ev
Wez 2016/06/22 19:54:23 Acknowledged.
+
// Returns true if there is a cache item |id|.
virtual bool Contains(const BlobId& id) const = 0;
« no previous file with comments | « no previous file | blimp/common/blob_cache/in_memory_blob_cache.h » ('j') | blimp/common/blob_cache/in_memory_blob_cache.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698