Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(535)

Unified Diff: chrome/browser/ui/panels/panel.cc

Issue 10260028: Update Panel mininimize/restore button on strip and expansion state change. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: feedback changes Created 8 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/panels/panel.h ('k') | chrome/browser/ui/panels/panel_browser_frame_view.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..3b6ed1aea2707815320509ab4c8389432547262f 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::UpdateMinimizeRestoreButtonVisibility() {
+ native_panel_->UpdatePanelMinimizeRestoreButtonVisibility();
+}
+
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));
+ UpdateMinimizeRestoreButtonVisibility();
content::NotificationService::current()->Notify(
chrome::NOTIFICATION_PANEL_CHANGED_EXPANSION_STATE,
« no previous file with comments | « chrome/browser/ui/panels/panel.h ('k') | chrome/browser/ui/panels/panel_browser_frame_view.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698