Don't expand path to use long names in CreateTemporaryFileInDir.

base/file_util_win.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/file_util.h"
 
 #include <windows.h>
 #include <psapi.h>
 #include <shellapi.h>
 #include <shlobj.h>
@@ skipping 375 matching lines @@
                               FilePath* temp_file) {
   base::ThreadRestrictions::AssertIOAllowed();
 
   wchar_t temp_name[MAX_PATH + 1];
 
   if (!GetTempFileName(dir.value().c_str(), L"", 0, temp_name)) {
     DPLOG(WARNING) << "Failed to get temporary file name in " << dir.value();
     return false;
   }
 
-  DWORD path_len = GetLongPathName(temp_name, temp_name, MAX_PATH);
-  if (path_len > MAX_PATH + 1 || path_len == 0) {
-    DPLOG(WARNING) << "Failed to get long path name for " << temp_name;
-    return false;
-  }
-
-  std::wstring temp_file_str;
-  temp_file_str.assign(temp_name, path_len);
-  *temp_file = FilePath(temp_file_str);
+  *temp_file = FilePath(temp_name);

[cpu_(ooo_6.6-7.5), 2012/11/09 17:21:47]

In principle I agree with you assessment but I would still like to have the
expanded path whenever the system allows it. So I think is best to keep the call
but don't make it fatal, and also remove the log statement.

   return true;
 }
 
 bool CreateTemporaryDirInDir(const FilePath& base_dir,
                              const FilePath::StringType& prefix,
                              FilePath* new_dir) {
   base::ThreadRestrictions::AssertIOAllowed();
 
   FilePath path_to_create;
   srand(static_cast<uint32>(time(NULL)));
@@ skipping 547 matching lines @@
   HANDLE cp = GetCurrentProcess();
   if (::GetMappedFileNameW(cp, file_view, mapped_file_path, kMaxPathLength)) {
     *nt_path = FilePath(mapped_file_path);
     success = true;
   }
   ::UnmapViewOfFile(file_view);
   return success;
 }
 
 }  // namespace file_util