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..099a8962dd449d2f1ba120656fdf238dde43f50e 100644
--- a/storage/browser/blob/blob_data_builder.cc
+++ b/storage/browser/blob/blob_data_builder.cc
@@ -25,7 +25,6 @@ void BlobDataBuilder::AppendFile(const base::FilePath& file_path,
                                  uint64_t offset,
                                  uint64_t length,
                                  const base::Time& expected_modification_time) {
-  DCHECK(length > 0);

[michaeln, 2015/02/25 20:35:43]

Out of curiosity, if (!length) return goes here instead, do tests pass?

[cmumford, 2015/02/25 22:37:29]

Yes, the tests do pass. Does that give you enough confidence to want that
change?

[michaeln, 2015/02/26 02:05:37]

A little uncomfortable without a better understanding of the purpose of the
release callback IDB has hooked up to the file.

   scoped_ptr<DataElement> element(new DataElement());
   element->SetToFilePathRange(file_path, offset, length,
                               expected_modification_time);