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

Unified Diff: chrome/browser/chromeos/login/screen_manager.cc

Issue 2739073002: cros: Use ScreenManager as a component instead of deriving WizardController from it (Closed)
Patch Set: Address comment Created 3 years, 9 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: chrome/browser/chromeos/login/screen_manager.cc
diff --git a/chrome/browser/chromeos/login/screen_manager.cc b/chrome/browser/chromeos/login/screen_manager.cc
index 8457c5a667114757f4e2400c5e5c3ad654f67ed6..23ee4fa69a51c63838b266737e9ff123675d47f7 100644
--- a/chrome/browser/chromeos/login/screen_manager.cc
+++ b/chrome/browser/chromeos/login/screen_manager.cc
@@ -4,22 +4,24 @@
#include "chrome/browser/chromeos/login/screen_manager.h"
+#include "base/memory/ptr_util.h"
+#include "chrome/browser/chromeos/login/wizard_controller.h"
+
namespace chromeos {
-ScreenManager::ScreenManager() {
-}
+ScreenManager::ScreenManager(WizardController* wizard_controller)
+ : wizard_controller_(wizard_controller) {}
-ScreenManager::~ScreenManager() {
-}
+ScreenManager::~ScreenManager() {}
BaseScreen* ScreenManager::GetScreen(OobeScreen screen) {
auto iter = screens_.find(screen);
if (iter != screens_.end())
return iter->second.get();
- BaseScreen* result = CreateScreen(screen);
+ BaseScreen* result = wizard_controller_->CreateScreen(screen);
DCHECK(result) << "Can not create screen named " << GetOobeScreenName(screen);
- screens_[screen] = make_linked_ptr(result);
+ screens_[screen] = base::WrapUnique(result);
return result;
}
« no previous file with comments | « chrome/browser/chromeos/login/screen_manager.h ('k') | chrome/browser/chromeos/login/screens/network_screen_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698