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

Unified Diff: apps/shell/browser/shell_browser_main_parts.cc

Issue 308783002: Removes the dependency to WMTestHelper from app shell. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix Created 6 years, 7 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: apps/shell/browser/shell_browser_main_parts.cc
diff --git a/apps/shell/browser/shell_browser_main_parts.cc b/apps/shell/browser/shell_browser_main_parts.cc
index b409f1775c301e8401a440f69483a8c107c9a79f..3dbfb7c06f8404489ece945195571b66a5913791 100644
--- a/apps/shell/browser/shell_browser_main_parts.cc
+++ b/apps/shell/browser/shell_browser_main_parts.cc
@@ -81,8 +81,8 @@ void ShellBrowserMainParts::PreMainMessageLoopRun() {
// Initialize our "profile" equivalent.
browser_context_.reset(new ShellBrowserContext);
- desktop_controller_.reset(new ShellDesktopController);
- desktop_controller_->GetWindowTreeHost()->AddObserver(this);
+ desktop_controller_.reset(browser_main_delegate_->CreateDesktopController());
+ desktop_controller_->CreateRootWindow();
// NOTE: Much of this is culled from chrome/test/base/chrome_test_suite.cc
// TODO(jamescook): Initialize chromeos::UserManager.
@@ -136,7 +136,6 @@ void ShellBrowserMainParts::PostMainMessageLoopRun() {
extensions_browser_client_.reset();
browser_context_.reset();
- desktop_controller_->GetWindowTreeHost()->RemoveObserver(this);
desktop_controller_.reset();
}
@@ -147,13 +146,6 @@ void ShellBrowserMainParts::PostDestroyThreads() {
#endif
}
-void ShellBrowserMainParts::OnHostCloseRequested(
- const aura::WindowTreeHost* host) {
- desktop_controller_->CloseAppWindow();
- base::MessageLoop::current()->PostTask(FROM_HERE,
- base::MessageLoop::QuitClosure());
-}
-
void ShellBrowserMainParts::CreateExtensionSystem() {
DCHECK(browser_context_);
extension_system_ = static_cast<ShellExtensionSystem*>(

Powered by Google App Engine
This is Rietveld 408576698