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

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

Issue 9195003: Move IN_OVERFLOW from Panel::ExpansionState to new enum. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix per feedback Created 8 years, 11 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.cc ('k') | chrome/browser/ui/panels/panel_browser_view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/panels/panel_browser_frame_view.cc
diff --git a/chrome/browser/ui/panels/panel_browser_frame_view.cc b/chrome/browser/ui/panels/panel_browser_frame_view.cc
index 9e92767389b4ffb13c34db1b63966bed4c086b90..decb2e33970e5918b4f87727c1b77580ab75a9eb 100644
--- a/chrome/browser/ui/panels/panel_browser_frame_view.cc
+++ b/chrome/browser/ui/panels/panel_browser_frame_view.cc
@@ -494,7 +494,7 @@ void PanelBrowserFrameView::Layout() {
bool show_close_button = true;
bool show_settings_button = true;
bool show_title_label = true;
- if (panel_browser_view_->panel()->expansion_state() == Panel::IN_OVERFLOW) {
+ if (panel_browser_view_->panel()->layout_state() == Panel::IN_OVERFLOW) {
if (width() <= IconOnlyWidth()) {
show_close_button = false;
show_settings_button = false;
@@ -865,7 +865,7 @@ void PanelBrowserFrameView::UpdateSettingsButtonVisibility(
DCHECK(has_settings_button_);
// The settings button is not shown in the overflow state.
- if (panel_browser_view_->panel()->expansion_state() == Panel::IN_OVERFLOW)
+ if (panel_browser_view_->panel()->layout_state() == Panel::IN_OVERFLOW)
return;
bool is_settings_button_visible = focused || cursor_in_view;
« no previous file with comments | « chrome/browser/ui/panels/panel.cc ('k') | chrome/browser/ui/panels/panel_browser_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698