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

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 7c89a8a084e072c00d946a7efc815777d0c317f9..fd800d79a478b9006f6c05356160333596b8ff05 100644
--- a/chrome/browser/ui/panels/panel.cc
+++ b/chrome/browser/ui/panels/panel.cc
@@ -157,11 +157,6 @@ gfx::Size Panel::ClampSize(const gfx::Size& size) const {
   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;
@@ -174,17 +169,15 @@ void Panel::EnableResizeByMouse(bool enable) {
   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;
@@ -194,6 +187,7 @@ void Panel::SetExpansionState(ExpansionState new_state) {
 
   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,