[BlobAsync] Disk support for blob storage

storage/browser/blob/shareable_blob_data_item.h

Index: storage/browser/blob/shareable_blob_data_item.h
diff --git a/storage/browser/blob/shareable_blob_data_item.h b/storage/browser/blob/shareable_blob_data_item.h
index 4c2f9c1833262d18dfa82b622abdfef8585b9541..e4fd4c68839013a78898a80f08964a921f454bc0 100644
--- a/storage/browser/blob/shareable_blob_data_item.h
+++ b/storage/browser/blob/shareable_blob_data_item.h
@@ -23,22 +23,31 @@ class InternalBlobData;
 // The data in this class (the item) is immutable, but the item itself can be
 // swapped out with an item with the same data but a different backing (think
 // RAM vs file backed).
-class ShareableBlobDataItem : public base::RefCounted<ShareableBlobDataItem> {
+class ShareableBlobDataItem
+    : public base::RefCountedThreadSafe<ShareableBlobDataItem> {

[michaeln, 2016/07/14 01:44:43]

Why does this need to be a threadsafe refcount? If this really is needed, seems
like BlobDataItem might also need a threadsafe refcount.

[dmurph, 2016/07/15 20:18:15]

We pass this to a file thread in the BlobMemoryController to write data to
files. We don't grab references to the BlobDataItem, so we don't need that to be
thread safe refcounted.

[michaeln, 2016/08/15 22:44:42]

But if the Shareable item can be deref'd on the background thread, can it also
be deleted on the background thread? If it is, the ref on its Item data member
will be decremented on the background thread.

  public:
-  ShareableBlobDataItem(const std::string& blob_uuid,
-                        const scoped_refptr<BlobDataItem>& item);
+  ShareableBlobDataItem(scoped_refptr<BlobDataItem> item);
 
   const scoped_refptr<BlobDataItem>& item();
 
+  // This is a unique auto-incrementing id assigned to this item on
+  // construction. It is used to keep track of this item in an LRU data
+  // structure for eviction to disk.
+  uint64_t item_id() const { return item_id_; }
+
   base::hash_set<std::string>& referencing_blobs() {
     return referencing_blobs_;
   }
 
  private:
-  friend class base::RefCounted<ShareableBlobDataItem>;
+  friend class base::RefCountedThreadSafe<ShareableBlobDataItem>;
   friend class InternalBlobData;
+  friend class BlobMemoryController;
+  friend class BlobStorageContext;
   ~ShareableBlobDataItem();
 
+  // This is a unique identifier for this ShareableBlobDataItem.
+  uint64_t item_id_;
   scoped_refptr<BlobDataItem> item_;
 
   base::hash_set<std::string> referencing_blobs_;