[BlobStorage] BlobMemoryController & tests

storage/browser/blob/internal_blob_data.cc

Index: storage/browser/blob/internal_blob_data.cc
diff --git a/storage/browser/blob/internal_blob_data.cc b/storage/browser/blob/internal_blob_data.cc
index 115c6a65e1f45265dff05d18054bf404455ecbfa..f52d41c639f98ddc4326aac5de8fbab03e194aad 100644
--- a/storage/browser/blob/internal_blob_data.cc
+++ b/storage/browser/blob/internal_blob_data.cc
@@ -58,7 +58,7 @@ InternalBlobData::items() const {
 void InternalBlobData::RemoveBlobFromShareableItems(
     const std::string& blob_uuid) {
   for (auto& data_item : items_) {
-    data_item->referencing_blobs().erase(blob_uuid);
+    data_item->referencing_blobs_mutable()->erase(blob_uuid);

[michaeln, 2016/10/11 20:55:24]

Would it be less error prone to "maybe free" each item when its collection of
referencing blobs becomes empty?

[dmurph, 2016/10/13 00:39:31]

Yeah. Done.

   }
 }