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

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

Issue 8538032: Keep panel manager internal titlebar state consistent with changes to titlebars. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 1 month 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_browser_view_browsertest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/panels/panel_manager.cc
diff --git a/chrome/browser/ui/panels/panel_manager.cc b/chrome/browser/ui/panels/panel_manager.cc
index e37eb43181267a871c5def44dc8c1f1920f06a94..7dd71557b56c6b4af00ae2b303ac8928c780484c 100644
--- a/chrome/browser/ui/panels/panel_manager.cc
+++ b/chrome/browser/ui/panels/panel_manager.cc
@@ -420,6 +420,10 @@ bool PanelManager::ShouldBringUpTitlebars(int mouse_x, int mouse_y) const {
}
void PanelManager::BringUpOrDownTitlebars(bool bring_up) {
+ if (are_titlebars_up_ == bring_up)
+ return;
+ are_titlebars_up_ = bring_up;
+
int task_delay_milliseconds = 0;
// If the auto-hiding bottom bar exists, delay the action until the bottom
@@ -557,9 +561,6 @@ BrowserWindow* PanelManager::GetNextBrowserWindowToActivate(
void PanelManager::OnMouseMove(const gfx::Point& mouse_position) {
bool bring_up_titlebars = ShouldBringUpTitlebars(mouse_position.x(),
mouse_position.y());
- if (are_titlebars_up_ == bring_up_titlebars)
- return;
- are_titlebars_up_ = bring_up_titlebars;
BringUpOrDownTitlebars(bring_up_titlebars);
}
« no previous file with comments | « chrome/browser/ui/panels/panel_browser_view_browsertest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698