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

Side by Side Diff: ash/common/shelf/wm_shelf.cc

Issue 2267803003: ash: Remove unnecessary WmShelf::SchedulePaint() method (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "ash/common/shelf/wm_shelf.h" 5 #include "ash/common/shelf/wm_shelf.h"
6 #include "ash/common/shelf/wm_shelf_observer.h" 6 #include "ash/common/shelf/wm_shelf_observer.h"
7 #include "ash/common/wm_lookup.h" 7 #include "ash/common/wm_lookup.h"
8 #include "ash/shelf/shelf.h" 8 #include "ash/shelf/shelf.h"
9 #include "ash/shelf/shelf_layout_manager.h" 9 #include "ash/shelf/shelf_layout_manager.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 87 matching lines...) Expand 10 before | Expand all | Expand 10 after
98 } 98 }
99 99
100 WmDimmerView* WmShelf::CreateDimmerView(bool disable_animations_for_test) { 100 WmDimmerView* WmShelf::CreateDimmerView(bool disable_animations_for_test) {
101 return nullptr; 101 return nullptr;
102 } 102 }
103 103
104 bool WmShelf::IsDimmed() const { 104 bool WmShelf::IsDimmed() const {
105 return shelf_layout_manager_->shelf_widget()->GetDimsShelf(); 105 return shelf_layout_manager_->shelf_widget()->GetDimsShelf();
106 } 106 }
107 107
108 void WmShelf::SchedulePaint() {
109 // Can be called during shutdown if the overflow bubble is visible.
110 if (shelf_)
111 shelf_->SchedulePaint();
112 }
113
114 bool WmShelf::IsVisible() const { 108 bool WmShelf::IsVisible() const {
115 return shelf_->IsVisible(); 109 return shelf_->IsVisible();
116 } 110 }
117 111
118 void WmShelf::UpdateVisibilityState() { 112 void WmShelf::UpdateVisibilityState() {
119 if (shelf_layout_manager_) 113 if (shelf_layout_manager_)
120 shelf_layout_manager_->UpdateVisibilityState(); 114 shelf_layout_manager_->UpdateVisibilityState();
121 } 115 }
122 116
123 ShelfVisibilityState WmShelf::GetVisibilityState() const { 117 ShelfVisibilityState WmShelf::GetVisibilityState() const {
(...skipping 75 matching lines...) Expand 10 before | Expand all | Expand 10 after
199 193
200 void WmShelf::OnBackgroundUpdated(ShelfBackgroundType background_type, 194 void WmShelf::OnBackgroundUpdated(ShelfBackgroundType background_type,
201 BackgroundAnimatorChangeType change_type) { 195 BackgroundAnimatorChangeType change_type) {
202 if (background_type == GetBackgroundType()) 196 if (background_type == GetBackgroundType())
203 return; 197 return;
204 FOR_EACH_OBSERVER(WmShelfObserver, observers_, 198 FOR_EACH_OBSERVER(WmShelfObserver, observers_,
205 OnBackgroundTypeChanged(background_type, change_type)); 199 OnBackgroundTypeChanged(background_type, change_type));
206 } 200 }
207 201
208 } // namespace ash 202 } // namespace ash
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698