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

Unified Diff: storage/browser/blob/blob_async_builder_host.h

Issue 1846363002: [BlobAsync] Adding better error reporting and some new tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: comments Created 4 years, 8 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: storage/browser/blob/blob_async_builder_host.h
diff --git a/storage/browser/blob/blob_async_builder_host.h b/storage/browser/blob/blob_async_builder_host.h
index 0b7fff5f7ffbff1358272df7c29b1d6b83d1152f..628bffa994382da7b69944d8fa707bd9ec25e983 100644
--- a/storage/browser/blob/blob_async_builder_host.h
+++ b/storage/browser/blob/blob_async_builder_host.h
@@ -109,7 +109,8 @@ class STORAGE_EXPORT BlobAsyncBuilderHost {
// This clears this object of pending construction. It also handles marking
// blobs that haven't been fully constructed as broken in the context if there
- // are any references being held by anyone.
+ // are any references being held by anyone. We know that they're being used
+ // by someone else if they still exist in the context.
void CancelAll(BlobStorageContext* context);
size_t blob_building_count() const { return async_blob_map_.size(); }
@@ -178,7 +179,8 @@ class STORAGE_EXPORT BlobAsyncBuilderHost {
// complete the blob and erase our internal state.
void ReferencedBlobFinished(const std::string& uuid,
base::WeakPtr<BlobStorageContext> context,
- bool construction_success);
+ bool construction_success,
+ IPCBlobCreationCancelCode reason);
// This finishes creating the blob in the context, decrements blob references
// that we were holding during construction, and erases our state.
« no previous file with comments | « content/browser/blob_storage/blob_dispatcher_host_unittest.cc ('k') | storage/browser/blob/blob_async_builder_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698