Truncate the download file name if it exceeds the filesystem limit.

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>
 #include <time.h>
 
+#include <algorithm>
 #include <limits>
 #include <string>
 
 #include "base/file_path.h"
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/process_util.h"
 #include "base/string_util.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/threading/thread_restrictions.h"
@@ skipping 918 matching lines @@
   bool success = false;
   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;
 }
 
+int GetMaximumPathComponentLength(const FilePath& path) {
+  base::ThreadRestrictions::AssertIOAllowed();
+
+  wchar_t volume_path[MAX_PATH];
+  if (!GetVolumePathNameW(path.NormalizePathSeparators().value().c_str(),
+                          volume_path,
+                          MAX_PATH)) {

[Mark Mentovai, 2013/02/15 13:49:42]

arraysize(volume_path)

[kinaba, 2013/02/18 03:30:21]

Done.

+    return -1;
+  }
+
+  DWORD max_length = 0;
+  if (!GetVolumeInformationW(volume_path, NULL, 0, NULL, &max_length, NULL,
+                             NULL, 0)) {
+    return -1;
+  }
+
+  // path.Append(name).value().size() must be less than MAX_PATH.
+  size_t prefix = path.StripTrailingSeparators().value().size() + 1;
+  int whole_path_limit = std::max(0, MAX_PATH - 1 - static_cast<int>(prefix));

[Mark Mentovai, 2013/02/15 13:49:42]

So one +1 is for the separator and one -1 is to keep it < MAX_PATH? It’s
confusing to do this in two places and on two lines without a comment.

[Mark Mentovai, 2013/02/15 13:49:42]

You might not be able to get rid of all of the static_casts, but maybe you can
get away with calling prefix an “int” to get rid of the cast on this line.

[kinaba, 2013/02/18 03:30:21]

Added some comments for clarification.

[kinaba, 2013/02/18 03:30:21]

The expression assigned to |prefix| has type size_t so calling it will just move
the static_cast to there. As far as I could come up, the current form is the
simplest (but not quire sure).

+  return std::min(whole_path_limit, static_cast<int>(max_length));
+}
+
 }  // namespace file_util