Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(63)

Side by Side Diff: third_party/WebKit/Source/platform/clipboard/ClipboardUtilitiesPosix.cpp

Issue 2674953003: Only generate suggested filenames when actually dragging an image. (Closed)
Patch Set: rebase Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 /*
2 * Copyright (C) 2009 Apple Inc. All rights reserved.
3 * Copyright (C) 2009 Google Inc.
4 *
5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions
7 * are met:
8 * 1. Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright
11 * notice, this list of conditions and the following disclaimer in the
12 * documentation and/or other materials provided with the distribution.
13 *
14 * THIS SOFTWARE IS PROVIDED BY APPLE COMPUTER, INC. ``AS IS'' AND ANY
15 * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
16 * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
17 * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE COMPUTER, INC. OR
18 * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
19 * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
20 * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
21 * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
22 * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
23 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
24 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
25 */
26
27 #include "platform/clipboard/ClipboardUtilities.h"
28
29 #include "wtf/text/WTFString.h"
30
31 namespace blink {
32
33 // On POSIX systems, the typical filename length limit is 255 character units.
34 // HFS+'s limit is actually 255 Unicode characters using Apple's modification of
35 // Normzliation Form D, but the differences aren't really worth dealing with
36 // here.
37 static const unsigned maxFilenameLength = 255;
38
39 static bool isInvalidFileCharacter(UChar c) {
40 // HFS+ disallows '/' and Linux systems also disallow null. For sanity's sake
41 // we'll also disallow control characters.
42 return c < ' ' || c == 0x7F || c == '/';
43 }
44
45 void validateFilename(String& name, String& extension) {
46 // Remove any invalid file system characters, especially "/".
47 name = name.removeCharacters(&isInvalidFileCharacter);
48 extension = extension.removeCharacters(&isInvalidFileCharacter);
49
50 // Remove a ridiculously-long extension.
51 if (extension.length() >= maxFilenameLength)
52 extension = String();
53
54 // Truncate an overly-long filename, reserving one character for a dot.
55 name.truncate(maxFilenameLength - extension.length() - 1);
56 }
57
58 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698