Fix panel showing logic when Chrome enters the fullscreen mode

chrome/browser/ui/panels/panel.cc

 // Copyright (c) 2012 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 "base/message_loop.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
@@ skipping 240 matching lines @@
   bounds.set_x(bounds.right() - full_size_.width());
   bounds.set_size(full_size_);
   return bounds;
 }
 
 gfx::Rect Panel::GetBounds() const {
   return native_panel_->GetPanelBounds();
 }
 
 void Panel::Show() {
-  if (manager()->display_settings_provider()->is_full_screen() || !collection_)
-    return;
-
   native_panel_->ShowPanel();
 }
 
 void Panel::Hide() {
   // Not implemented.
 }
 
 void Panel::ShowInactive() {
+  // Only if the panel is created without initial focus, we delay its showing

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

The description of the bug repeats this statement, but it's unclear why. Could
you update description to explain the rationale and maybe even add a comment
here to do the same?

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

Removed this comment and put back the removed code in Panel::Show as discussed.

+  // until the fullscreen mode is exited.
   if (manager()->display_settings_provider()->is_full_screen() || !collection_)
     return;
 
   native_panel_->ShowPanelInactive();
 }
 
 // Close() may be called multiple times if the panel window is not ready to
 // close on the first attempt.
 void Panel::Close() {
   native_panel_->ClosePanel();
@@ skipping 593 matching lines @@
 
 // static
 void Panel::FormatTitleForDisplay(string16* title) {
   size_t current_index = 0;
   size_t match_index;
   while ((match_index = title->find(L'\n', current_index)) != string16::npos) {
     title->replace(match_index, 1, string16());
     current_index = match_index;
   }
 }