Enforce WARN_UNUSED_RESULT attribute on Windows

chrome/installer/util/delete_tree_work_item.cc

Index: chrome/installer/util/delete_tree_work_item.cc
diff --git a/chrome/installer/util/delete_tree_work_item.cc b/chrome/installer/util/delete_tree_work_item.cc
index 9cc47ede0058478f9fa718f5d36be482358cfab5..8a94c44517eee4edd74362665caa182411b6f26c 100644
--- a/chrome/installer/util/delete_tree_work_item.cc
+++ b/chrome/installer/util/delete_tree_work_item.cc
@@ -67,7 +67,6 @@ bool DeleteTreeWorkItem::Do() {
         PLOG(ERROR) << "Could not back up " << key_file.value()
                     << " to directory " << backup.path().value();
         abort = true;
-        backup.Delete();

[Lei Zhang, 2015/10/07 16:28:09]

Why not Delete() ?

[dcheng, 2015/10/07 16:41:34]

It's a ScopedTmpDir, so it self-deletes anyway when it exits the scope (which
already emits a warning message if the delete fails: see
https://code.google.com/p/chromium/codesearch#chromium/src/base/files/scoped_...)

[Lei Zhang, 2015/10/07 20:33:11]

But |backup| is a reference to |key_backup_paths_[i]|.

[dcheng, 2015/10/08 03:48:07]

Done. Thanks for pointing this out.

       }
     }
     if (!abort) {