Set last-error code in DeleteFileAndRecordMetrics if deletion failed

Bug: 1133844
Change-Id: I5c0c5baa21e591db236cebba80697eb75f9612ef
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2450073
Commit-Queue: Michael Li <michli@microsoft.com>
Reviewed-by: Greg Thompson <grt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#814422}

base/files/file_util_win.cc

@@ -352,6 +352,9 @@ DWORD DoDeleteFile(const FilePath& path, bool recursive) {
              : ReturnLastErrorOrSuccessOnNotFound();
 }
 
+// Deletes the file/directory at |path| (recursively if |recursive| and |path|
+// names a directory), returning true on success. Sets the Windows last-error
+// code and returns false on failure.
 bool DeleteFileAndRecordMetrics(const FilePath& path, bool recursive) {
   static constexpr char kRecursive[] = "DeleteFile.Recursive";
   static constexpr char kNonRecursive[] = "DeleteFile.NonRecursive";
@@ -369,6 +372,8 @@ bool DeleteFileAndRecordMetrics(const FilePath& path, bool recursive) {
     return true;
 
   RecordFilesystemError(operation, error);
+
+  ::SetLastError(error);
   return false;
 }