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

Side by Side Diff: services/ui/ws/user_display_manager.cc

Issue 2646863002: Fix unused lambda captures in //chrome, //services/ui and //ui. (Closed)
Patch Set: Created 3 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 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 "services/ui/ws/user_display_manager.h" 5 #include "services/ui/ws/user_display_manager.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "services/ui/display/screen_manager.h" 9 #include "services/ui/display/screen_manager.h"
10 #include "services/ui/ws/display.h" 10 #include "services/ui/ws/display.h"
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
43 void UserDisplayManager::OnFrameDecorationValuesChanged() { 43 void UserDisplayManager::OnFrameDecorationValuesChanged() {
44 if (!got_valid_frame_decorations_) { 44 if (!got_valid_frame_decorations_) {
45 got_valid_frame_decorations_ = true; 45 got_valid_frame_decorations_ = true;
46 display_manager_observers_.ForAllPtrs([this]( 46 display_manager_observers_.ForAllPtrs([this](
47 mojom::DisplayManagerObserver* observer) { CallOnDisplays(observer); }); 47 mojom::DisplayManagerObserver* observer) { CallOnDisplays(observer); });
48 return; 48 return;
49 } 49 }
50 50
51 std::vector<mojom::WsDisplayPtr> displays = GetAllDisplays(); 51 std::vector<mojom::WsDisplayPtr> displays = GetAllDisplays();
52 display_manager_observers_.ForAllPtrs( 52 display_manager_observers_.ForAllPtrs(
53 [this, &displays](mojom::DisplayManagerObserver* observer) { 53 [&displays](mojom::DisplayManagerObserver* observer) {
54 observer->OnDisplaysChanged(CloneDisplays(displays)); 54 observer->OnDisplaysChanged(CloneDisplays(displays));
55 }); 55 });
56 } 56 }
57 57
58 void UserDisplayManager::AddDisplayManagerBinding( 58 void UserDisplayManager::AddDisplayManagerBinding(
59 mojo::InterfaceRequest<mojom::DisplayManager> request) { 59 mojo::InterfaceRequest<mojom::DisplayManager> request) {
60 display_manager_bindings_.AddBinding(this, std::move(request)); 60 display_manager_bindings_.AddBinding(this, std::move(request));
61 } 61 }
62 62
63 void UserDisplayManager::OnDisplayUpdate(Display* display) { 63 void UserDisplayManager::OnDisplayUpdate(Display* display) {
(...skipping 105 matching lines...) Expand 10 before | Expand all | Expand 10 after
169 mojom::DisplayManagerObserver* observer) { 169 mojom::DisplayManagerObserver* observer) {
170 // TODO(kylechar): Pass internal display id to clients here. 170 // TODO(kylechar): Pass internal display id to clients here.
171 observer->OnDisplays( 171 observer->OnDisplays(
172 GetAllDisplays(), 172 GetAllDisplays(),
173 display::ScreenManager::GetInstance()->GetPrimaryDisplayId(), 173 display::ScreenManager::GetInstance()->GetPrimaryDisplayId(),
174 display::kInvalidDisplayId); 174 display::kInvalidDisplayId);
175 } 175 }
176 176
177 } // namespace ws 177 } // namespace ws
178 } // namespace ui 178 } // namespace ui
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698