[BlobAsync] Adding better error reporting and some new tests.

storage/browser/blob/blob_data_handle.cc

 // Copyright (c) 2013 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.
 
 #include "storage/browser/blob/blob_data_handle.h"
 
 #include <stdint.h>
 
 #include <memory>
 
@@ skipping 57 matching lines @@
     const std::string& content_type,
     const std::string& content_disposition,
     BlobStorageContext* context)
     : uuid_(uuid),
       content_type_(content_type),
       content_disposition_(content_disposition),
       context_(context->AsWeakPtr()) {
   context_->IncrementBlobRefCount(uuid);
 }
 
-void BlobDataHandle::BlobDataHandleShared::RunOnConstructionComplete(
-    const base::Callback<void(bool)>& done) {
-  if (!context_.get()) {
-    done.Run(false);
-    return;
-  }
-  context_->RunOnConstructionComplete(uuid_, done);
-}
-
 std::unique_ptr<BlobReader> BlobDataHandle::CreateReader(
     FileSystemContext* file_system_context,
     base::SequencedTaskRunner* file_task_runner) const {
   return std::unique_ptr<BlobReader>(new BlobReader(
       this, std::unique_ptr<BlobReader::FileStreamReaderProvider>(
                 new FileStreamReaderProviderImpl(file_system_context)),
       file_task_runner));
 }
 
-std::unique_ptr<BlobDataSnapshot>
-BlobDataHandle::BlobDataHandleShared::CreateSnapshot() const {
-  if (!context_.get())
-    return nullptr;
-  return context_->CreateSnapshot(uuid_);
-}
-
 BlobDataHandle::BlobDataHandleShared::~BlobDataHandleShared() {
   if (context_.get())
     context_->DecrementBlobRefCount(uuid_);
 }
 
 BlobDataHandle::BlobDataHandle(const std::string& uuid,
                                const std::string& content_type,
                                const std::string& content_disposition,
                                BlobStorageContext* context,
                                base::SequencedTaskRunner* io_task_runner)
@@ skipping 28 matching lines @@
 }
 
 bool BlobDataHandle::IsBroken() const {
   DCHECK(io_task_runner_->RunsTasksOnCurrentThread());
   if (!shared_->context_)
     return true;
   return shared_->context_->IsBroken(shared_->uuid_);
 }
 
 void BlobDataHandle::RunOnConstructionComplete(
-    const base::Callback<void(bool)>& done) {
+    const BlobConstructedCallback& done) {
   DCHECK(io_task_runner_->RunsTasksOnCurrentThread());
-  shared_->RunOnConstructionComplete(done);
+  if (!shared_->context_.get()) {
+    done.Run(false, IPCBlobCreationCancelCode::UNKNOWN);
+    return;
+  }
+  shared_->context_->RunOnConstructionComplete(shared_->uuid_, done);
 }
 
 std::unique_ptr<BlobDataSnapshot> BlobDataHandle::CreateSnapshot() const {
   DCHECK(io_task_runner_->RunsTasksOnCurrentThread());
-  return shared_->CreateSnapshot();
+  if (!shared_->context_.get())
+    return nullptr;
+  return shared_->context_->CreateSnapshot(shared_->uuid_);
 }
 
 const std::string& BlobDataHandle::uuid() const {
   return shared_->uuid_;
 }
 
 const std::string& BlobDataHandle::content_type() const {
   return shared_->content_type_;
 }
 
 const std::string& BlobDataHandle::content_disposition() const {
   return shared_->content_disposition_;
 }
 
 }  // namespace storage