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

Unified Diff: services/ui/ws/display_manager.cc

Issue 2692863009: Remove usage of ws::Display in ws::UserDisplayManager. (Closed)
Patch Set: Fix mash_browser_tests. Created 3 years, 10 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 | « services/ui/display/screen_manager_stub_internal.cc ('k') | services/ui/ws/test_utils.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: services/ui/ws/display_manager.cc
diff --git a/services/ui/ws/display_manager.cc b/services/ui/ws/display_manager.cc
index 8489c4b620d09764b501536dd8ca1f7cb8a8778d..2cbaa8ebadeae044443241addcdebbd60547d170 100644
--- a/services/ui/ws/display_manager.cc
+++ b/services/ui/ws/display_manager.cc
@@ -45,7 +45,7 @@ UserDisplayManager* DisplayManager::GetUserDisplayManager(
const UserId& user_id) {
if (!user_display_managers_.count(user_id)) {
user_display_managers_[user_id] =
- base::MakeUnique<UserDisplayManager>(this, window_server_, user_id);
+ base::MakeUnique<UserDisplayManager>(window_server_, user_id);
}
return user_display_managers_[user_id].get();
}
@@ -69,7 +69,7 @@ void DisplayManager::DestroyDisplay(Display* display) {
pending_displays_.erase(display);
} else {
for (const auto& pair : user_display_managers_)
- pair.second->OnWillDestroyDisplay(display);
+ pair.second->OnWillDestroyDisplay(display->GetId());
DCHECK(displays_.count(display));
displays_.erase(display);
@@ -100,7 +100,7 @@ std::set<const Display*> DisplayManager::displays() const {
void DisplayManager::OnDisplayUpdate(Display* display) {
for (const auto& pair : user_display_managers_)
- pair.second->OnDisplayUpdate(display);
+ pair.second->OnDisplayUpdate(display->ToDisplay());
}
Display* DisplayManager::GetDisplayContaining(const ServerWindow* window) {
« no previous file with comments | « services/ui/display/screen_manager_stub_internal.cc ('k') | services/ui/ws/test_utils.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698