Use __func__ instead of __FUNCTION__.

chrome/installer/util/delete_after_reboot_helper.cc

Index: chrome/installer/util/delete_after_reboot_helper.cc
diff --git a/chrome/installer/util/delete_after_reboot_helper.cc b/chrome/installer/util/delete_after_reboot_helper.cc
index c9cf0fe783f56796ee07eccf46067eed48f2697c..59f4508d9557e2a76c152ec9536df24a92b7e184 100644
--- a/chrome/installer/util/delete_after_reboot_helper.cc
+++ b/chrome/installer/util/delete_after_reboot_helper.cc
@@ -252,7 +252,7 @@ base::FilePath GetShortPathName(const base::FilePath& path) {
                                   MAX_PATH);
   DWORD last_error = ::GetLastError();

[grt (UTC plus 2), 2016/07/20 06:18:51]

while you're at it, could you remove this line and change the one below to:
  DPLOG_IF(...) << __func__;
this will result in a human-readable error message in the log

[Peter Kasting, 2016/07/20 06:27:22]

That last error value is used in the condition for the DLOG.  Does calling
GetLastError() reset the error code?  If so, the change you suggest wouldn't be
safe.  Otherwise I suppose we could do:

  DPLOG_IF(WARNING, length == 0 && GetLastError() != ERROR_PATH_NOT_FOUND)
      << __func__;

(Although personally I wish this would do DVPLOG_IF(1, ...) instead of using
DPLOG().)

[grt (UTC plus 2), 2016/07/20 06:32:47]

Ah, I just saw "DLOG" and "gle" in the message without looking too closely.
Apologies.
No.
I think that's safe. Feel free to leave this out if you prefer not to make a
behavior change with this substitution.

[Peter Kasting, 2016/07/20 06:35:51]

I don't mind at all, it's more convenient than doing it in a separate CL.

   DLOG_IF(WARNING, length == 0 && last_error != ERROR_PATH_NOT_FOUND)
-      << __FUNCTION__ << " gle=" << last_error;
+      << __func__ << " gle=" << last_error;
   if ((length == 0) || (length > MAX_PATH)) {
     // GetShortPathName fails if the path is no longer present or cannot be
     // put in the size buffer provided.  Instead of returning an empty string,