[Native File System] Atomic Writes review changes

Adding review changes from https://chromium-review.googlesource.com/c/chromium/src/+/1692325
I prematurely hit the submit button.

Bug: 968550
Change-Id: I52b85c70d5b47845a60398d1c453a118e1be5a6a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1726660Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
Commit-Queue: Olivier Yiptong <oyiptong@chromium.org>
Cr-Commit-Position: refs/heads/master@{#682557}

content/browser/native_file_system/native_file_system_file_handle_impl.cc

@@ -156,8 +156,7 @@ void NativeFileSystemFileHandleImpl::CreateFileWriterImpl(
   auto base_swap_path =
       base::FilePath(url().path()).AddExtensionASCII(".crswap");
   CreateSwapFile(
-      /*count=*/0, std::move(base_swap_path), keep_existing_data,
-      std::move(callback));
+      /*count=*/0, base_swap_path, keep_existing_data, std::move(callback));
 }
 
 void NativeFileSystemFileHandleImpl::CreateSwapFile(

content/browser/native_file_system/native_file_system_file_writer_impl.h

@@ -89,6 +89,9 @@ class CONTENT_EXPORT NativeFileSystemFileWriterImpl
     kClosed,
   };
   bool is_closed() const { return state_ != State::kOpen; }
+  // Returns whether the File Writer is in a state where any files can be
+  // deleted. We do not want to delete the files if there are clean-up
+  // operations in-flight.
   bool can_purge() const {
     return state_ == State::kOpen || state_ == State::kCloseError;
   }

third_party/blink/renderer/modules/native_file_system/native_file_system_writer.cc

@@ -285,7 +285,8 @@ void NativeFileSystemWriter::CloseComplete(
   }
   file_ = nullptr;
   pending_operation_ = nullptr;
-  // Closes the mojo pipe. Subsequent operations will fail.
+  // We close the mojo pipe because we intend this writer to be discarded after
+  // close. Subsequent operations will fail.
   mojo_ptr_ = nullptr;
 }