Initial version of Blimp BlobCache.

blimp/common/blob_cache/infinite_blob_cache.h

+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef BLIMP_COMMON_BLOB_CACHE_INFINITE_BLOB_CACHE_H_
+#define BLIMP_COMMON_BLOB_CACHE_INFINITE_BLOB_CACHE_H_
+
+#include <map>
+#include <string>
+#include <vector>
+
+#include "base/macros.h"
+#include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"

[Kevin M, 2016/04/13 23:29:52]

Remove

[nyquist, 2016/04/14 19:33:22]

Done.

+#include "blimp/common/blimp_common_export.h"
+#include "blimp/common/blob_cache/blob_cache.h"
+
+namespace blimp {
+
+// InfiniteBlobCache provides an in-memory implementation of BlobCache that
+// never evicts items.
+class BLIMP_COMMON_EXPORT InfiniteBlobCache : public BlobCache {

[vmpstr, 2016/04/13 23:48:02]

Since there's only one implementation of BlobCache.. and that implementation is
just std::map, do you think it might be worth it to just use an std::map
wherever you'd use this BlobCache?

That's something that we can refactor later if there will be more than one
cache... There are also some base/ caches like MRUCache that you might use...

Basically, I'm not sure I would add another cache class since I don't see any
benefit over existing maps.

[nyquist, 2016/04/14 19:33:22]

The plan is definitely later to use a different implementation, and the
very-very-soon plan is to use this implementation to flesh out the rest of the
BlobChannel feature so we can get closer to an asynchronous world of
displaying/sending images. So I'd like to keep this in here for now.

+ public:
+  InfiniteBlobCache();
+  ~InfiniteBlobCache() override;
+
+  // BlobCache implementation.
+  bool Contains(const std::string& id) const override;
+  void Put(const std::string& id,
+           scoped_refptr<RefCountedVector> data) override;
+  scoped_refptr<RefCountedVector> Get(const std::string& id) const override;
+
+ private:
+  typedef std::map<std::string, scoped_refptr<RefCountedVector>> Cache;
+  Cache map_;
+
+  DISALLOW_COPY_AND_ASSIGN(InfiniteBlobCache);
+};
+
+}  // namespace blimp
+
+#endif  // BLIMP_COMMON_BLOB_CACHE_INFINITE_BLOB_CACHE_H_