Mac: Added a delay before panels go to minimized state from title-only state.

chrome/browser/ui/panels/panel.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/panels/panel.h"
 
 #include "base/logging.h"
 #include "chrome/browser/extensions/extension_prefs.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "content/browser/renderer_host/render_view_host.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/panels/native_panel.h"
 #include "chrome/browser/ui/panels/panel_manager.h"
 #include "chrome/browser/ui/window_sizer.h"
 #include "chrome/browser/web_applications/web_app.h"
 #include "chrome/common/extensions/extension.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "content/common/view_messages.h"
+#include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_source.h"
 #include "content/public/browser/notification_types.h"
 #include "ui/gfx/rect.h"
 
 // static
 const Extension* Panel::GetExtensionFromBrowser(Browser* browser) {
   // Find the extension. When we create a panel from an extension, the extension
   // ID is passed as the app name to the Browser.
   ExtensionService* extension_service =
       browser->GetProfile()->GetExtensionService();
@@ skipping 74 matching lines @@
   gfx::Rect bounds = native_panel_->GetPanelBounds();
   bounds.set_y(bottom - height);
   bounds.set_height(height);
   SetPanelBounds(bounds);
 
   manager()->OnPanelExpansionStateChanged(old_state, new_state);
 
   // The minimized panel should not get the focus.
   if (expansion_state_ == MINIMIZED)
     Deactivate();
+
+  content::NotificationService::current()->Notify(

[jianli, 2011/10/28 21:31:33]

Probably we do not need this notification since RunAllPending() seems to be
enough.

[Dmitry Titov, 2011/10/29 00:04:56]

Discussed offline. Decided to go with regualr pattern of waiting for explicit
notification rather then depend on RunAllPending().

+      chrome::NOTIFICATION_PANEL_CHANGED_EXPANSION_STATE,
+      content::Source<Panel>(this),
+      content::NotificationService::NoDetails());
 }
 
 bool Panel::ShouldBringUpTitlebar(int mouse_x, int mouse_y) const {
   // Skip the expanded panel.
   if (expansion_state_ == EXPANDED)
     return false;
 
   // If the panel is showing titlebar only, we want to keep it up when it is
   // being dragged.
   if (expansion_state_ == TITLE_ONLY && manager()->is_dragging_panel())
@@ skipping 464 matching lines @@
   RequestRenderViewHostToDisableScrollbars(GetRenderViewHost());
 }
 
 Browser* Panel::browser() const {
   return native_panel_->GetPanelBrowser();
 }
 
 void Panel::DestroyBrowser() {
   native_panel_->DestroyPanelBrowser();
 }