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

Unified Diff: ash/mus/bridge/wm_shell_mus.cc

Issue 2115663002: Folds methods in WmShellCommon to WmShell (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: moar Created 4 years, 6 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
Index: ash/mus/bridge/wm_shell_mus.cc
diff --git a/ash/mus/bridge/wm_shell_mus.cc b/ash/mus/bridge/wm_shell_mus.cc
index baa07bbd89e0230a0ddfb54aff374c0ce0c40ecd..a655c7327dea14ff89100921b5101473d695066a 100644
--- a/ash/mus/bridge/wm_shell_mus.cc
+++ b/ash/mus/bridge/wm_shell_mus.cc
@@ -14,7 +14,6 @@
#include "ash/common/wm/mru_window_tracker.h"
#include "ash/common/wm/window_resizer.h"
#include "ash/common/wm_activation_observer.h"
-#include "ash/common/wm_shell_common.h"
#include "ash/mus/bridge/wm_root_window_controller_mus.h"
#include "ash/mus/bridge/wm_window_mus.h"
#include "ash/mus/container_ids.h"
@@ -97,8 +96,7 @@ WmShellMus::WmShellMus(::mus::WindowTreeClient* client)
client_->AddObserver(this);
WmShell::Set(this);
- wm_shell_common_.reset(new WmShellCommon);
- wm_shell_common_->CreateMruWindowTracker();
+ CreateMruWindowTracker();
accessibility_delegate_.reset(new DefaultAccessibilityDelegate);
SetSystemTrayDelegate(base::WrapUnique(new DefaultSystemTrayDelegate));
@@ -107,7 +105,7 @@ WmShellMus::WmShellMus(::mus::WindowTreeClient* client)
WmShellMus::~WmShellMus() {
DeleteSystemTrayDelegate();
DeleteWindowSelectorController();
- wm_shell_common_->DeleteMruWindowTracker();
+ DeleteMruWindowTracker();
RemoveClientObserver();
WmShell::Set(nullptr);
}
@@ -151,10 +149,6 @@ WmRootWindowControllerMus* WmShellMus::GetRootWindowControllerWithDisplayId(
return nullptr;
}
-MruWindowTracker* WmShellMus::GetMruWindowTracker() {
- return wm_shell_common_->mru_window_tracker();
-}
-
WmWindow* WmShellMus::NewContainerWindow() {
return WmWindowMus::Get(client_->NewWindow());
}
@@ -240,13 +234,12 @@ WmShellMus::CreateMaximizeModeEventHandler() {
}
void WmShellMus::OnOverviewModeStarting() {
- FOR_EACH_OBSERVER(ShellObserver, *wm_shell_common_->shell_observers(),
+ FOR_EACH_OBSERVER(ShellObserver, *shell_observers(),
OnOverviewModeStarting());
}
void WmShellMus::OnOverviewModeEnded() {
- FOR_EACH_OBSERVER(ShellObserver, *wm_shell_common_->shell_observers(),
- OnOverviewModeEnded());
+ FOR_EACH_OBSERVER(ShellObserver, *shell_observers(), OnOverviewModeEnded());
}
AccessibilityDelegate* WmShellMus::GetAccessibilityDelegate() {
@@ -273,14 +266,6 @@ void WmShellMus::RemoveDisplayObserver(WmDisplayObserver* observer) {
NOTIMPLEMENTED();
}
-void WmShellMus::AddShellObserver(ShellObserver* observer) {
- wm_shell_common_->AddShellObserver(observer);
-}
-
-void WmShellMus::RemoveShellObserver(ShellObserver* observer) {
- wm_shell_common_->RemoveShellObserver(observer);
-}
-
void WmShellMus::AddPointerWatcher(views::PointerWatcher* watcher) {
// TODO(jamescook): Move PointerWatcherDelegateMus to //ash/mus and use here.
NOTIMPLEMENTED();

Powered by Google App Engine
This is Rietveld 408576698