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

Side by Side 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, 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "apps/shell/browser/shell_browser_main_parts.h" 5 #include "apps/shell/browser/shell_browser_main_parts.h"
6 6
7 #include "apps/shell/browser/shell_browser_context.h" 7 #include "apps/shell/browser/shell_browser_context.h"
8 #include "apps/shell/browser/shell_browser_main_delegate.h" 8 #include "apps/shell/browser/shell_browser_main_delegate.h"
9 #include "apps/shell/browser/shell_desktop_controller.h" 9 #include "apps/shell/browser/shell_desktop_controller.h"
10 #include "apps/shell/browser/shell_extension_system.h" 10 #include "apps/shell/browser/shell_extension_system.h"
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 // TODO(jamescook): Initialize chromeos::CrosSettings here? 74 // TODO(jamescook): Initialize chromeos::CrosSettings here?
75 75
76 // Return no error. 76 // Return no error.
77 return 0; 77 return 0;
78 } 78 }
79 79
80 void ShellBrowserMainParts::PreMainMessageLoopRun() { 80 void ShellBrowserMainParts::PreMainMessageLoopRun() {
81 // Initialize our "profile" equivalent. 81 // Initialize our "profile" equivalent.
82 browser_context_.reset(new ShellBrowserContext); 82 browser_context_.reset(new ShellBrowserContext);
83 83
84 desktop_controller_.reset(new ShellDesktopController); 84 desktop_controller_.reset(browser_main_delegate_->CreateDesktopController());
85 desktop_controller_->GetWindowTreeHost()->AddObserver(this); 85 desktop_controller_->CreateRootWindow();
86 86
87 // NOTE: Much of this is culled from chrome/test/base/chrome_test_suite.cc 87 // NOTE: Much of this is culled from chrome/test/base/chrome_test_suite.cc
88 // TODO(jamescook): Initialize chromeos::UserManager. 88 // TODO(jamescook): Initialize chromeos::UserManager.
89 net_log_.reset(new content::ShellNetLog("app_shell")); 89 net_log_.reset(new content::ShellNetLog("app_shell"));
90 90
91 extensions_client_.reset(new ShellExtensionsClient()); 91 extensions_client_.reset(new ShellExtensionsClient());
92 extensions::ExtensionsClient::Set(extensions_client_.get()); 92 extensions::ExtensionsClient::Set(extensions_client_.get());
93 93
94 extensions_browser_client_.reset( 94 extensions_browser_client_.reset(
95 new extensions::ShellExtensionsBrowserClient(browser_context_.get())); 95 new extensions::ShellExtensionsBrowserClient(browser_context_.get()));
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 void ShellBrowserMainParts::PostMainMessageLoopRun() { 129 void ShellBrowserMainParts::PostMainMessageLoopRun() {
130 browser_main_delegate_->Shutdown(); 130 browser_main_delegate_->Shutdown();
131 131
132 BrowserContextDependencyManager::GetInstance()->DestroyBrowserContextServices( 132 BrowserContextDependencyManager::GetInstance()->DestroyBrowserContextServices(
133 browser_context_.get()); 133 browser_context_.get());
134 extension_system_ = NULL; 134 extension_system_ = NULL;
135 extensions::ExtensionsBrowserClient::Set(NULL); 135 extensions::ExtensionsBrowserClient::Set(NULL);
136 extensions_browser_client_.reset(); 136 extensions_browser_client_.reset();
137 browser_context_.reset(); 137 browser_context_.reset();
138 138
139 desktop_controller_->GetWindowTreeHost()->RemoveObserver(this);
140 desktop_controller_.reset(); 139 desktop_controller_.reset();
141 } 140 }
142 141
143 void ShellBrowserMainParts::PostDestroyThreads() { 142 void ShellBrowserMainParts::PostDestroyThreads() {
144 #if defined(OS_CHROMEOS) 143 #if defined(OS_CHROMEOS)
145 network_controller_.reset(); 144 network_controller_.reset();
146 chromeos::DBusThreadManager::Shutdown(); 145 chromeos::DBusThreadManager::Shutdown();
147 #endif 146 #endif
148 } 147 }
149 148
150 void ShellBrowserMainParts::OnHostCloseRequested(
151 const aura::WindowTreeHost* host) {
152 desktop_controller_->CloseAppWindow();
153 base::MessageLoop::current()->PostTask(FROM_HERE,
154 base::MessageLoop::QuitClosure());
155 }
156
157 void ShellBrowserMainParts::CreateExtensionSystem() { 149 void ShellBrowserMainParts::CreateExtensionSystem() {
158 DCHECK(browser_context_); 150 DCHECK(browser_context_);
159 extension_system_ = static_cast<ShellExtensionSystem*>( 151 extension_system_ = static_cast<ShellExtensionSystem*>(
160 ExtensionSystem::Get(browser_context_.get())); 152 ExtensionSystem::Get(browser_context_.get()));
161 extension_system_->InitForRegularProfile(true); 153 extension_system_->InitForRegularProfile(true);
162 } 154 }
163 155
164 } // namespace apps 156 } // namespace apps
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698