chromeos: Move EnableScreenLock to PowerManagerClient from PowerLibrary

chrome/browser/chromeos/system/screen_locker_settings.cc

Index: chrome/browser/chromeos/system/screen_locker_settings.cc
diff --git a/chrome/browser/chromeos/system/screen_locker_settings.cc b/chrome/browser/chromeos/system/screen_locker_settings.cc
new file mode 100644
index 0000000000000000000000000000000000000000..32e03b0a313b07e0ff55088859f07a87bc2184d2
--- /dev/null
+++ b/chrome/browser/chromeos/system/screen_locker_settings.cc
@@ -0,0 +1,44 @@
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/chromeos/system/touchpad_settings.h"
+
+#include "base/bind.h"
+#include "base/file_path.h"
+#include "base/file_util.h"
+#include "base/stringprintf.h"
+#include "chrome/browser/chromeos/system/runtime_environment.h"
+#include "content/public/browser/browser_thread.h"
+
+using content::BrowserThread;
+
+namespace {
+
+const char kLockOnIdleSuspendPath[] =
+    "/var/lib/power_manager/lock_on_idle_suspend";
+
+}  // namespace
+
+namespace chromeos {
+namespace system {
+namespace screen_locker_settings {
+
+void EnableScreenLock(bool enable) {
+  // Run this on the FILE thread.
+  if (!BrowserThread::CurrentlyOn(BrowserThread::FILE)) {
+    BrowserThread::PostTask(
+        BrowserThread::FILE, FROM_HERE,
+        base::Bind(&EnableScreenLock, enable));

[satorux1, 2011/11/16 23:58:04]

Please don't be call itself. Please create a separate function in anonymous
namespace like EnableScreenLockOnFileThread() and call it.

[satorux1, 2011/11/16 23:59:15]

Please also be sure to put

DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI))

at the beginning of this this function, and 

DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE)) in
EnableScreenLockOnFileThread()

[Simon Que, 2011/11/18 18:56:08]

Done.

[Simon Que, 2011/11/18 18:56:08]

Done.

+    return;
+  }
+
+  std::string config = base::StringPrintf("%d", enable);
+  file_util::WriteFile(FilePath(kLockOnIdleSuspendPath),

[satorux1, 2011/11/16 23:58:04]

We shouldn't write this file on linux desktop. Please use
system::runtime_eivnronment::IsRunningOnChromeOS() and write code for Linux
desktop, that does nothing.

[Simon Que, 2011/11/18 18:56:08]

Done.

+                       config.c_str(),
+                       config.size());
+}
+
+}  // namespace screen_locker_settings
+}  // namespace system
+}  // namespace chromeos