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

chrome/browser/ui/views/extensions/extension_dialog.cc

Index: chrome/browser/ui/views/extensions/extension_dialog.cc
diff --git a/chrome/browser/ui/views/extensions/extension_dialog.cc b/chrome/browser/ui/views/extensions/extension_dialog.cc
index 32bc344759030129f11a4c160ccd3ff75e0db5aa..e13fb9d78fb537f069f092cc7633020226deaa48 100644
--- a/chrome/browser/ui/views/extensions/extension_dialog.cc
+++ b/chrome/browser/ui/views/extensions/extension_dialog.cc
@@ -113,9 +113,6 @@ void ExtensionDialog::Close() {
   if (!window_)
     return;
 
-  if (observer_)
-    observer_->ExtensionDialogIsClosing(this);
-
   window_->Close();
   window_ = NULL;
 }
@@ -139,6 +136,12 @@ string16 ExtensionDialog::GetWindowTitle() const {
   return window_title_;
 }
 
+void ExtensionDialog::WindowClosing() {
+  // Cleanup the pending dialog list.

[flackr, 2011/11/28 19:47:06]

Correct me if I'm wrong, but your comment is specific to the file selector
dialog, right? I think the code is fairly self documenting, in that anything you
write wouldn't be any more specific than just reading that the observer is
notified of the window closing. So in this case remove the comment.

[bshe, 2011/11/28 22:26:50]

Done.
It is quite self documenting, so I will just remove it.

On 2011/11/28 19:47:06, flackr wrote:

+  if (observer_)
+    observer_->ExtensionDialogIsClosing(this);
+}
+
 void ExtensionDialog::DeleteDelegate() {
   // The window has finished closing.  Allow ourself to be deleted.
   Release();