diff options
| author | mukai@chromium.org <mukai@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-23 18:15:06 +0000 |
|---|---|---|
| committer | mukai@chromium.org <mukai@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-23 18:15:06 +0000 |
| commit | 12946d64538c583c4a1561fc6e64e43a128a832f (patch) | |
| tree | c8b1109d240174269a5fbe50edd95e448ce27389 /ash | |
| parent | 74ce7234c65015f91080d64f76a0116a8a9bb5d9 (diff) | |
| download | chromium_src-12946d64538c583c4a1561fc6e64e43a128a832f.zip chromium_src-12946d64538c583c4a1561fc6e64e43a128a832f.tar.gz chromium_src-12946d64538c583c4a1561fc6e64e43a128a832f.tar.bz2 | |
Moves calling OnAutoHideState to when exactly the state has changed.
This method is called when a window is maximized or fullscreen.
BUG=249509
R=harrym@chromium.org
TEST=manually
Review URL: https://chromiumcodereview.appspot.com/19968005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@213169 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ash')
| -rw-r--r-- | ash/shelf/shelf_layout_manager.cc | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/ash/shelf/shelf_layout_manager.cc b/ash/shelf/shelf_layout_manager.cc index e0d06d7..4fa302b 100644 --- a/ash/shelf/shelf_layout_manager.cc +++ b/ash/shelf/shelf_layout_manager.cc @@ -211,8 +211,6 @@ void ShelfLayoutManager::SetAutoHideBehavior(ShelfAutoHideBehavior behavior) { auto_hide_behavior_ = behavior; UpdateVisibilityState(); FOR_EACH_OBSERVER(ShelfLayoutManagerObserver, observers_, - OnAutoHideStateChanged(state_.auto_hide_state)); - FOR_EACH_OBSERVER(ShelfLayoutManagerObserver, observers_, OnAutoHideBehaviorChanged(root_window_, auto_hide_behavior_)); } @@ -333,17 +331,12 @@ void ShelfLayoutManager::UpdateAutoHideState() { if (auto_hide_state == SHELF_AUTO_HIDE_HIDDEN) { // Hides happen immediately. SetState(state_.visibility_state); - FOR_EACH_OBSERVER(ShelfLayoutManagerObserver, observers_, - OnAutoHideStateChanged(auto_hide_state)); } else { auto_hide_timer_.Stop(); auto_hide_timer_.Start( FROM_HERE, base::TimeDelta::FromMilliseconds(kAutoHideDelayMS), this, &ShelfLayoutManager::UpdateAutoHideStateNow); - FOR_EACH_OBSERVER(ShelfLayoutManagerObserver, observers_, - OnAutoHideStateChanged( - CalculateAutoHideState(state_.visibility_state))); } } else { auto_hide_timer_.Stop(); @@ -658,6 +651,11 @@ void ShelfLayoutManager::SetState(ShelfVisibilityState visibility_state) { UpdateHitTestBounds(); if (!delay_shelf_update) UpdateShelfBackground(change_type); + + if (old_state.auto_hide_state != state_.auto_hide_state) { + FOR_EACH_OBSERVER(ShelfLayoutManagerObserver, observers_, + OnAutoHideStateChanged(state_.auto_hide_state)); + } } void ShelfLayoutManager::StopAnimating() { |