Changeset 82914 in webkit

Timestamp:

Apr 5, 2011 2:06:45 AM (13 years ago)

Author:

commit-queue@webkit.org

Message:

2011-04-05 Sailesh Agrawal <​sail@chromium.org>

Reviewed by Darin Fisher.

Don't clear existing files if the user cancels the file chooser dialog.
​https://bugs.webkit.org/show_bug.cgi?id=55200

• src/WebFileChooserCompletionImpl.cpp: (WebKit::WebFileChooserCompletionImpl::didChooseFile):

Location:

trunk/Source/WebKit/chromium

Files:

• ChangeLog (modified) (1 diff)
• src/WebFileChooserCompletionImpl.cpp (modified) (1 diff)

trunk/Source/WebKit/chromium/ChangeLog

@@ +1 @@
+2011-04-05  Sailesh Agrawal  <sail@chromium.org>
+
+        Reviewed by Darin Fisher.
+
+        Don't clear existing files if the user cancels the file chooser dialog.
+        https://bugs.webkit.org/show_bug.cgi?id=55200
+
+        * src/WebFileChooserCompletionImpl.cpp:
+        (WebKit::WebFileChooserCompletionImpl::didChooseFile):
+
 2011-04-05  John Mellor  <johnme@chromium.org>
 

trunk/Source/WebKit/chromium/src/WebFileChooserCompletionImpl.cpp

@@ -47 +47 @@
     if (fileNames.size() == 1)
         m_fileChooser->chooseFile(fileNames[0]);
-    else {
-        // This clause handles a case of file_names.size()==0 too.
+    else if (fileNames.size() > 0) {
         Vector<WTF::String> paths;
         for (size_t i = 0; i < fileNames.size(); ++i)