Fix panel showing logic when Chrome enters the fullscreen mode

chrome/browser/ui/cocoa/panels/panel_cocoa.mm

Index: chrome/browser/ui/cocoa/panels/panel_cocoa.mm
diff --git a/chrome/browser/ui/cocoa/panels/panel_cocoa.mm b/chrome/browser/ui/cocoa/panels/panel_cocoa.mm
index 722d88ed24bbe7f2db2ddfa78a63d2fc9319d6c8..453fcab328279fcb890e36881914a0e82bcaa32e 100644
--- a/chrome/browser/ui/cocoa/panels/panel_cocoa.mm
+++ b/chrome/browser/ui/cocoa/panels/panel_cocoa.mm
@@ -210,8 +210,18 @@ void PanelCocoa::HandlePanelKeyboardEvent(
   [event_window redispatchKeyEvent:event.os_event];
 }
 
-void PanelCocoa::FullScreenModeChanged(
-    bool is_full_screen) {
+void PanelCocoa::FullScreenModeChanged(bool is_full_screen) {
+  if (!is_shown_) {
+    // If the panel window is not shown due to that a Chrome tab window is in
+    // fullscreen mode when the panel is being created, we need to show the
+    // panel window now. In addition, its titlebar needs to be updated since it
+    // is not done at the panel creation time.
+    if (!is_full_screen) {
+      ShowPanelInactive();
+      UpdatePanelTitleBar();
+    }
+    return;

[Dmitry Titov, 2013/03/14 01:10:25]

why this return? don't we need to do the controller_ call?

[jianli, 2013/03/14 19:11:48]

No, we do not need the call. If the panel window has not been shown yet, we will
show it when |is_full_screen| is true and we do not need to show it and call the
controller when it is still in full screen mode. Added comment for this.

+  }
   [controller_ fullScreenModeChanged:is_full_screen];
 }