Merge 87848 - 2011-06-01 Daniel Cheng <dcheng@chromium.org>

Source/WebCore/platform/chromium/ClipboardChromiumMac.cpp

 /*
  * Copyright (C) 2009 Apple Inc.  All rights reserved.
  * Copyright (C) 2009 Google Inc.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 10 matching lines @@
  * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
  * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 #include "ClipboardChromium.h"
 
 #include "ChromiumDataObject.h"
-#include "NotImplemented.h"
 
 namespace WebCore {
+    
+
+// Filename length and character-set limits vary by filesystem, and there's no
+// real way to tell what filesystem is in use. Since HFS+ is by far the most
+// common, we'll abide by its limits, which are 255 Unicode characters (slightly
+// simplified; really it uses Normalization Form D, but the differences aren't
+// really worth dealing with here.)
+static const unsigned MaxHFSFilenameLength = 255;
+
+
+static bool isInvalidFileCharacter(UChar c)
+{
+    // HFS+ basically allows anything but '/'. For sanity's sake we'll disallow
+    // control characters.
+    return c < ' ' || c == 0x7F || c == '/';
+}
 
 String ClipboardChromium::validateFileName(const String& title, ChromiumDataObject* dataObject)
 {
-    notImplemented();
-    return title;
+    // Remove any invalid file system characters, especially "/".
+    String result = title.removeCharacters(isInvalidFileCharacter);
+    String extension = dataObject->fileExtension().removeCharacters(&isInvalidFileCharacter);
+
+    // Remove a ridiculously-long extension.
+    if (extension.length() >= MaxHFSFilenameLength)
+        extension = "";
+    
+    // Truncate an overly-long filename.
+    int overflow = result.length() + extension.length() - MaxHFSFilenameLength;
+    if (overflow > 0) 
+        result.remove(result.length() - overflow, overflow);
+    
+    dataObject->setFileExtension(extension);
+    return result;
 }
 
 }  // namespace WebCore