I'm expanding this to fix a couple of bugs in determining the file name for...

I'm expanding this to fix a couple of bugs in determining the file name for
dragging images:

. It's possible for the extension to be empty, resulting in a bad file
  name, for example, 'foo.' or just '.'.
. We weren't restricting the size of the file to MAX_PATH.
. We weren't removing characters that are invalid for file system names.

It's possible to put this code up in OSExchangeData rather than
here, but it feels best to centralize it here so that we don't have to
worry about bad file names leaking into other code.

I tried hard to write a layout test for this, but it seems we don't
really use the file name in test shell so that I couldn't get
coverage.

BUG=7023
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=10206

[tony, 2009-02-20 00:22:09]

The code looks fine, but if we're going to validate the filename on windows, I
think we should validate on other platforms as well.  So can we add the validate
method to the class and have ClipboardChromiumWin.cpp etc.  You can leave the
mac/linuc implementations as identity stubs with a notImplemented() so we don't
forget on other platforms.

[tony, 2009-02-20 02:15:44]

Yes, this is what I was expecting.  LGTM!

http://codereview.chromium.org/20504/diff/7/8
File third_party/WebKit/WebCore/platform/chromium/ClipboardChromium.cpp (right):

http://codereview.chromium.org/20504/diff/7/8#newcode46
Line 46: #include <shlwapi.h>
Nit: can you remove this now?