Update Panel mininimize/restore button on strip and expansion state change.

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 "chrome/browser/ui/panels/docked_panel_strip.h"
 #include "chrome/browser/profiles/profile.h"
@@ skipping 139 matching lines @@
   // * cannot grow or shrink to go beyond [min_height, max_height]
   int new_height = size.height();
   if (new_height > max_size_.height())
     new_height = max_size_.height();
   if (new_height < min_size_.height())
     new_height = min_size_.height();
 
   return gfx::Size(new_width, new_height);
 }
 
-
-void Panel::SetAppIconVisibility(bool visible) {
-  native_panel_->SetPanelAppIconVisibility(visible);
-}
-
 void Panel::SetAlwaysOnTop(bool on_top) {
   if (always_on_top_ == on_top)
     return;
   always_on_top_ = on_top;
   native_panel_->SetPanelAlwaysOnTop(on_top);
 }
 
 void Panel::EnableResizeByMouse(bool enable) {
   DCHECK(native_panel_);
   native_panel_->EnableResizeByMouse(enable);
 }
 
+void Panel::UpdateTitlebarButtons() {

[Dmitry Titov, 2012/05/02 03:23:08]

Why not just keep the names here in sync if this is a simple delegate to the
native impl? Also, this method is not about all titlebar buttons.

[jennb, 2012/05/02 19:10:35]

Done.

+  native_panel_->UpdateMinimizeRestoreButtonVisibility();
+}
+
 void Panel::SetPreviewMode(bool in_preview) {
   DCHECK_NE(in_preview_mode_, in_preview);
   in_preview_mode_ = in_preview;
 }
 
-void  Panel::SetPanelStrip(PanelStrip* new_strip) {
-  panel_strip_ = new_strip;
-  if (panel_strip_ != NULL && initialized_)
-    native_panel_->PreventActivationByOS(panel_strip_->IsPanelMinimized(this));
-}
-
 void Panel::SetExpansionState(ExpansionState new_state) {
   if (expansion_state_ == new_state)
     return;
   expansion_state_ = new_state;
 
   manager()->OnPanelExpansionStateChanged(this);
 
   DCHECK(initialized_ && panel_strip_ != NULL);
   native_panel_->PreventActivationByOS(panel_strip_->IsPanelMinimized(this));
+  native_panel_->UpdateMinimizeRestoreButtonVisibility();
 
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_PANEL_CHANGED_EXPANSION_STATE,
       content::Source<Panel>(this),
       content::NotificationService::NoDetails());
 }
 
 bool Panel::IsDrawingAttention() const {
   return native_panel_->IsDrawingAttention();
 }
@@ skipping 556 matching lines @@
 
   if (modifier == panel::APPLY_TO_ALL)
     panel_strip_->RestoreAll();
   else
     Restore();
 }
 
 void Panel::DestroyBrowser() {
   native_panel_->DestroyPanelBrowser();
 }