Fix for cant reopen file dialog after close it by x button or alt+f4

chrome/browser/ui/views/select_file_dialog_extension.cc

Index: chrome/browser/ui/views/select_file_dialog_extension.cc
diff --git a/chrome/browser/ui/views/select_file_dialog_extension.cc b/chrome/browser/ui/views/select_file_dialog_extension.cc
index 132daa2e82868892c3bd928894395c6aea4b2d09..51e9f4433c91bef6843fea354214981ffce06f51 100644
--- a/chrome/browser/ui/views/select_file_dialog_extension.cc
+++ b/chrome/browser/ui/views/select_file_dialog_extension.cc
@@ -212,7 +212,7 @@ void SelectFileDialogExtension::SelectFileImpl(
   // possible.
   int32 tab_id = tab ? tab->restore_tab_helper()->session_id().id() : 0;
   if (PendingExists(tab_id))
-    return;
+    PendingDialog::GetInstance()->Remove(tab_id);

[Rick Byers, 2011/11/25 15:06:06]

I haven't looked at the code in detail, but doesn't this just circumvent what
PendingDialog was trying to do (make sure you never have two dialogs up at
once)?

[bshe, 2011/11/25 17:58:49]

I am not sure when and how a user attempts to open two dialogs in the same tab.
I was think maybe we can give user some sort of feedback (bring the previous one
up or delete previous one and create a new one or a warning dialog) when they
try to do so instead of just ignore. But I guess it depends on the specific
situation. Maybe ignore its exactly what we want. 

Also giving some sort of feedback may require some works, if the situation is
rare, it might not worth to spend time on it at least for now.

Anyway, I am working on making the close button behaves the same as cancel
button.
 

On 2011/11/25 15:06:06, Rick Byers wrote:

 
   FilePath virtual_path;
   if (!file_manager_util::ConvertFileToRelativeFileSystemPath(