IndexedDB: Fixed support for empty blobs.

storage/browser/blob/blob_data_builder.cc

Index: storage/browser/blob/blob_data_builder.cc
diff --git a/storage/browser/blob/blob_data_builder.cc b/storage/browser/blob/blob_data_builder.cc
index 51946c882f4e8dfec78e38fdfe4966c771695d7e..3199fcec70a574937407573eba2e871dcdd2c50a 100644
--- a/storage/browser/blob/blob_data_builder.cc
+++ b/storage/browser/blob/blob_data_builder.cc
@@ -15,7 +15,6 @@ BlobDataBuilder::~BlobDataBuilder() {
 }
 
 void BlobDataBuilder::AppendData(const char* data, size_t length) {
-  DCHECK(length > 0);

[michaeln, 2015/02/25 00:51:59]

Are we also hitting this dcheck? Can you reduce it down to just relaxing those
that we trip on.

[cmumford, 2015/02/25 18:09:49]

Looks like only one DCHECK was actually hit by the new tests so keeping the
other three.

[jsbell, 2015/02/25 19:53:26]

s/keeping/removing/ ?

   scoped_ptr<DataElement> element(new DataElement());
   element->SetToBytes(data, length);
   items_.push_back(new BlobDataItem(element.Pass()));
@@ -25,7 +24,6 @@ void BlobDataBuilder::AppendFile(const base::FilePath& file_path,
                                  uint64_t offset,
                                  uint64_t length,
                                  const base::Time& expected_modification_time) {
-  DCHECK(length > 0);
   scoped_ptr<DataElement> element(new DataElement());
   element->SetToFilePathRange(file_path, offset, length,
                               expected_modification_time);
@@ -36,7 +34,6 @@ void BlobDataBuilder::AppendFile(const base::FilePath& file_path,
 void BlobDataBuilder::AppendBlob(const std::string& uuid,
                                  uint64_t offset,
                                  uint64_t length) {
-  DCHECK_GT(length, 0ul);
   scoped_ptr<DataElement> element(new DataElement());
   element->SetToBlobRange(uuid, offset, length);
   items_.push_back(new BlobDataItem(element.Pass()));
@@ -53,7 +50,6 @@ void BlobDataBuilder::AppendFileSystemFile(
     uint64_t offset,
     uint64_t length,
     const base::Time& expected_modification_time) {
-  DCHECK(length > 0);
   scoped_ptr<DataElement> element(new DataElement());
   element->SetToFileSystemUrlRange(url, offset, length,
                                    expected_modification_time);