Use completion callback instead of NOTREACHED for invalid offsets

This allows to simplify Write methods in NativeFileSystemFileHandleImpl.

Bug: 957214
Change-Id: I4e1ba58a7f904ebe5f60d355f20cdd6c5cb497bb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645782Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
Commit-Queue: Eero Häkkinen <eero.hakkinen@intel.com>
Cr-Commit-Position: refs/heads/master@{#666624}

storage/browser/fileapi/sandbox_file_stream_writer.cc

@@ -136,10 +136,9 @@ void SandboxFileStreamWriter::DidCreateSnapshotFile(
   }
   file_size_ = file_info.size;
   if (initial_offset_ > file_size_) {
-    LOG(ERROR) << initial_offset_ << ", " << file_size_;
-    // This shouldn't happen as long as we check offset in the renderer.
-    NOTREACHED();
-    initial_offset_ = file_size_;
+    // We should not be writing pass the end of the file.
+    std::move(callback).Run(net::ERR_REQUEST_RANGE_NOT_SATISFIABLE);
+    return;
   }
   DCHECK(!file_writer_.get());