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

Unified Diff: chrome/browser/chromeos/settings/cros_settings.cc

Issue 14200028: Make CrosSettings and DeviceSettingsService non Lazy instances (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 7 years, 8 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/settings/cros_settings.cc
diff --git a/chrome/browser/chromeos/settings/cros_settings.cc b/chrome/browser/chromeos/settings/cros_settings.cc
index f0944071549fe51582752db5f5d1e1b7cb4a25eb..2e7e3565e774809510b39491c84fa70b97358c41 100644
--- a/chrome/browser/chromeos/settings/cros_settings.cc
+++ b/chrome/browser/chromeos/settings/cros_settings.cc
@@ -6,10 +6,11 @@
#include "base/bind.h"
#include "base/command_line.h"
-#include "base/lazy_instance.h"
+#include "base/logging.h"
#include "base/stl_util.h"
#include "base/string_util.h"
#include "base/values.h"
+#include "chrome/browser/chromeos/login/user_manager.h"
#include "chrome/browser/chromeos/settings/device_settings_provider.h"
#include "chrome/browser/chromeos/settings/device_settings_service.h"
#include "chrome/browser/chromeos/settings/kiosk_app_local_settings.h"
@@ -24,12 +25,30 @@
namespace chromeos {
-static base::LazyInstance<CrosSettings> g_cros_settings =
- LAZY_INSTANCE_INITIALIZER;
+static CrosSettings* g_cros_settings = NULL;
+// static
+void CrosSettings::Initialize() {
+ CHECK(!g_cros_settings);
+ g_cros_settings = new CrosSettings();
+}
+
+// static
+bool CrosSettings::IsInitialized() {
+ return g_cros_settings;
+}
+
+// static
+void CrosSettings::Shutdown() {
+ CHECK(g_cros_settings);
+ delete g_cros_settings;
+ g_cros_settings = NULL;
+}
+
+// static
CrosSettings* CrosSettings::Get() {
- // TODO(xiyaun): Use real stuff when underlying libcros is ready.
- return g_cros_settings.Pointer();
+ CHECK(g_cros_settings);
+ return g_cros_settings;
}
bool CrosSettings::IsCrosSettings(const std::string& path) {
@@ -310,4 +329,21 @@ void CrosSettings::FireObservers(const std::string& path) {
}
}
+ScopedTestCrosSettings::ScopedTestCrosSettings()
+ : initialized_device_settings_service_(false) {
+ if (!DeviceSettingsService::IsInitialized()) {
+ DeviceSettingsService::Initialize();
+ initialized_device_settings_service_ = true;
+ }
+ CrosSettings::Initialize();
+}
+
+ScopedTestCrosSettings::~ScopedTestCrosSettings() {
+ // UserManager holds a CrosSettings*, so ensure that it is destroyed.
+ UserManager::Set(NULL);
+ CrosSettings::Shutdown();
+ if (initialized_device_settings_service_)
+ DeviceSettingsService::Shutdown();
+}
+
} // namespace chromeos
« no previous file with comments | « chrome/browser/chromeos/settings/cros_settings.h ('k') | chrome/browser/chromeos/settings/device_settings_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698