Reland: Implement correct Panel closing sequence for Mac.

chrome/browser/ui/panels/panel.cc

Index: chrome/browser/ui/panels/panel.cc
diff --git a/chrome/browser/ui/panels/panel.cc b/chrome/browser/ui/panels/panel.cc
index 450eee5cd5c74ed7f53f57809f5fffd22a15de81..8c0cf87d9aa5501e7bc4e006d3743d9aa27d19af 100644
--- a/chrome/browser/ui/panels/panel.cc
+++ b/chrome/browser/ui/panels/panel.cc
@@ -87,7 +87,13 @@ void Panel::SetBounds(const gfx::Rect& bounds) {
 // close on the first attempt.
 void Panel::Close() {
   native_panel_->ClosePanel();
+
+// TODO(dimich): we need to only remove the panel when and if it actually

[jennb, 2011/08/19 21:46:12]

Even if the panel isn't closing immediately, can't the panel be hidden offscreen
and removed from panel manager immediately?

[Dmitry Titov, 2011/08/22 18:34:56]

See comment at the top.

+// closes. It is now only implemented on Mac (unless it's synchronous on other
+// platforms).
+#if !defined(OS_MACOSX)
   manager()->Remove(this);
+#endif
 }
 
 void Panel::Activate() {