Move Mac LaunchOnStartup enable/disable to File thread

chrome/browser/background_mode_manager_linux.cc

Index: chrome/browser/background_mode_manager_linux.cc
diff --git a/chrome/browser/background_mode_manager_linux.cc b/chrome/browser/background_mode_manager_linux.cc
new file mode 100644
index 0000000000000000000000000000000000000000..17c2ed4c2ad691293f71cd69e50667f801809d9d
--- /dev/null
+++ b/chrome/browser/background_mode_manager_linux.cc
@@ -0,0 +1,117 @@
+// Copyright (c) 2010 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 <unistd.h>
+
+#include "app/l10n_util.h"
+#include "base/command_line.h"
+#include "base/environment.h"
+#include "base/file_path.h"
+#include "base/file_util.h"
+#include "base/logging.h"
+#include "base/nix/xdg_util.h"
+#include "base/task.h"
+#include "chrome/app/chrome_command_ids.h"

[Andrew T Wilson (Slow), 2010/12/04 18:49:48]

I don't think this is needed any more (chrome_command_ids)

[The wrong rickcam account, 2010/12/07 22:59:57]

Done.

+#include "chrome/browser/background_mode_manager.h"
+#include "chrome/browser/browser_thread.h"
+#include "chrome/browser/gtk/gtk_util.h"
+#include "chrome/browser/prefs/pref_service.h"
+#include "chrome/browser/shell_integration.h"
+#include "chrome/common/chrome_switches.h"
+#include "chrome/common/chrome_version_info.h"
+#include "chrome/common/pref_names.h"
+#include "grit/generated_resources.h"
+
+class DisableLaunchOnStartupTask : public Task {
+ public:
+  virtual void Run();

[Andrew T Wilson (Slow), 2010/12/04 18:49:48]

BTW, one thing I sometimes do for these private classes is just put the
implementation inline, especially since there's only one method. It's OK as-is
though.

One other question - should these be put in an anonymous namespace?

[The wrong rickcam account, 2010/12/07 22:59:57]

Two-part answer:

(1) I'd like to leave the definitions as is.  When there's only one member
function AND that definition is concise, I will also sometimes do this.  If
either one is meaty, I like to keep them separate.  In this case, there's the
additional bit where I think that it's a Good Thing to have Disable... and
Enable... follow the same patterns.  EnableLaunchOnStartupTask::Run, with a
30-line body, tips the scale for me.

(2) Yes.  I sprinked anonymous namespaces around the Task class definitions
throughout background_mode_manager*.*, expanding existing anonymous namespace
regions where it seemed appropriate.

+};
+
+class EnableLaunchOnStartupTask : public Task {
+ public:
+  virtual void Run();
+};
+
+static const FilePath::CharType kAutostart[] = "autostart";
+static const FilePath::CharType kConfig[] = ".config";
+static const char kXdgConfigHome[] = "XDG_CONFIG_HOME";
+
+namespace {
+
+FilePath GetAutostartDirectory(base::Environment* environment) {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
+  FilePath result =
+      base::nix::GetXDGDirectory(environment, kXdgConfigHome, kConfig);
+  result = result.Append(kAutostart);
+  return result;
+}
+
+FilePath GetAutostartFilename(base::Environment* environment) {
+  FilePath directory = GetAutostartDirectory(environment);
+  return directory.Append(ShellIntegration::GetDesktopName(environment));
+}
+
+}  // namespace
+
+void BackgroundModeManager::EnableLaunchOnStartup(bool should_launch) {
+  // This functionality is only defined for default profile, currently.
+  if (CommandLine::ForCurrentProcess()->HasSwitch(switches::kUserDataDir))
+    return;
+  if (should_launch) {
+    BrowserThread::PostTask(BrowserThread::FILE, FROM_HERE,
+                            new EnableLaunchOnStartupTask());
+  } else {
+    BrowserThread::PostTask(BrowserThread::FILE, FROM_HERE,
+                            new DisableLaunchOnStartupTask());
+  }
+}
+
+void DisableLaunchOnStartupTask::Run() {
+  scoped_ptr<base::Environment> environment(base::Environment::Create());
+  if (!file_util::Delete(GetAutostartFilename(environment.get()), false)) {
+    LOG(WARNING) << "Failed to deregister launch on login.";

[Andrew T Wilson (Slow), 2010/12/04 18:49:48]

I might add a NOT_REACHED here as I think we'd like to know if this is routinely
getting hit, whereas most people never look at their logs.

[The wrong rickcam account, 2010/12/07 22:59:57]

Done.  I replaced LOG(...) with NOTREACHED() throughout
background_mode_manager*.*

+  }
+}
+
+// TODO(rickcam): Bug 56280: Share implementation with ShellIntegration
+void EnableLaunchOnStartupTask::Run() {
+  scoped_ptr<base::Environment> environment(base::Environment::Create());
+  scoped_ptr<chrome::VersionInfo> version_info(new chrome::VersionInfo());
+  FilePath autostart_directory = GetAutostartDirectory(environment.get());
+  FilePath autostart_file = GetAutostartFilename(environment.get());
+  if (!file_util::DirectoryExists(autostart_directory) &&
+      !file_util::CreateDirectory(autostart_directory)) {
+    LOG(WARNING)
+      << "Failed to register launch on login.  No autostart directory.";
+    return;
+  }
+  std::string wrapper_script;
+  if (!environment->GetVar("CHROME_WRAPPER", &wrapper_script)) {
+    LOG(WARNING)
+      << "Failed to register launch on login.  CHROME_WRAPPER not set.";
+    return;
+  }
+  std::string autostart_file_contents =
+      "[Desktop Entry]\n"
+      "Type=Application\n"
+      "Terminal=false\n"
+      "Exec=" + wrapper_script +
+      " --enable-background-mode --no-startup-window\n"
+      "Name=" + version_info->Name() + "\n";
+  std::string::size_type content_length = autostart_file_contents.length();
+  if (file_util::WriteFile(autostart_file, autostart_file_contents.c_str(),
+                           content_length) !=
+      static_cast<int>(content_length)) {
+    LOG(WARNING) << "Failed to register launch on login.  Failed to write "
+                 << autostart_file.value();
+    file_util::Delete(GetAutostartFilename(environment.get()), false);
+  }
+}
+
+string16 BackgroundModeManager::GetPreferencesMenuLabel() {
+  string16 result = gtk_util::GetStockPreferencesMenuLabel();
+  if (!result.empty())
+    return result;
+  return l10n_util::GetStringUTF16(IDS_PREFERENCES);
+}