Linux: allow using GNOME Keyring and KWallet for password storage via a command line flag.

chrome/browser/profile.cc

 // 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 "chrome/browser/profile.h"
 
 #include "app/resource_bundle.h"
 #include "app/theme_provider.h"
 #include "base/command_line.h"
 #include "base/env_var.h"
@@ skipping 266 matching lines @@
 // static
 URLRequestContextGetter* Profile::GetDefaultRequestContext() {
   return default_request_context_;
 }
 
 #if defined(OS_WIN)
 #include "chrome/browser/password_manager/password_store_win.h"
 #elif defined(OS_MACOSX)
 #include "chrome/browser/keychain_mac.h"
 #include "chrome/browser/password_manager/password_store_mac.h"
-#elif defined(OS_POSIX)
-// Temporarily disabled while we figure some stuff out.
-// http://code.google.com/p/chromium/issues/detail?id=12351
-// #include "chrome/browser/password_manager/password_store_gnome.h"
-// #include "chrome/browser/password_manager/password_store_kwallet.h"
+#elif defined(OS_POSIX) && !defined(OS_CHROMEOS)
+#include "base/xdg_util.h"
+#include "chrome/browser/password_manager/native_backend_gnome_x.h"
+#include "chrome/browser/password_manager/native_backend_kwallet_x.h"
+#include "chrome/browser/password_manager/password_store_x.h"
 #endif
 
 ////////////////////////////////////////////////////////////////////////////////
 //
 // OffTheRecordProfileImpl is a profile subclass that wraps an existing profile
 // to make it suitable for the off the record mode.
 //
 ////////////////////////////////////////////////////////////////////////////////
 class OffTheRecordProfileImpl : public Profile,
                                 public NotificationObserver {
@@ skipping 1051 matching lines @@
   if (!login_db->Init(login_db_file_path)) {
     LOG(ERROR) << "Could not initialize login database.";
     delete login_db;
     return;
   }
 #if defined(OS_WIN)
   ps = new PasswordStoreWin(login_db, this,
                             GetWebDataService(Profile::IMPLICIT_ACCESS));
 #elif defined(OS_MACOSX)
   ps = new PasswordStoreMac(new MacKeychain(), login_db);
-#elif defined(OS_POSIX)
-  // TODO(evanm): implement "native" password management.
-  // This bug describes the issues.
-  // http://code.google.com/p/chromium/issues/detail?id=12351
+#elif defined(OS_CHROMEOS)
+  // For now, we use PasswordStoreDefault. We might want to make a native
+  // backend for PasswordStoreX (see below) in the future though.
   ps = new PasswordStoreDefault(login_db, this,
                                 GetWebDataService(Profile::IMPLICIT_ACCESS));
+#elif defined(OS_POSIX)
+  // On POSIX systems, we try to use the "native" password management system of
+  // the desktop environment currently running, allowing GNOME Keyring in XFCE.
+  // (In all cases we fall back on the default store in case of failure.)
+  base::DesktopEnvironment desktop_env;
+  std::wstring store_type = CommandLine::ForCurrentProcess()->GetSwitchValue(
+      switches::kPasswordStore);
+  if (store_type == L"kwallet") {
+    desktop_env = base::DESKTOP_ENVIRONMENT_KDE4;
+  } else if (store_type == L"gnome") {
+    desktop_env = base::DESKTOP_ENVIRONMENT_GNOME;
+  } else if (store_type == L"detect") {
+    scoped_ptr<base::EnvVarGetter> env_getter(base::EnvVarGetter::Create());
+    desktop_env = base::GetDesktopEnvironment(env_getter.get());
+    LOG(INFO) << "Password storage detected desktop environment: " <<
+              base::GetDesktopEnvironmentName(desktop_env);
+  } else {
+    // TODO(mdm): If the flag is not given, or has an unknown value, use the
+    // default store for now. Once we're confident in the other stores, we can
+    // default to detecting the desktop environment instead.
+    desktop_env = base::DESKTOP_ENVIRONMENT_OTHER;
+  }
+
+  scoped_ptr<PasswordStoreX::NativeBackend> backend;
+  if (desktop_env == base::DESKTOP_ENVIRONMENT_KDE4) {
+    // KDE3 didn't use DBus, which our KWallet store uses.
+    LOG(INFO) << "Trying KWallet for password storage.";
+    backend.reset(new NativeBackendKWallet());
+    if (backend->Init())
+      LOG(INFO) << "Using KWallet for password storage.";
+    else
+      backend.reset();
+  } else if (desktop_env == base::DESKTOP_ENVIRONMENT_GNOME ||
+             desktop_env == base::DESKTOP_ENVIRONMENT_XFCE) {
+    LOG(INFO) << "Trying GNOME keyring for password storage.";
+    backend.reset(new NativeBackendGnome());
+    if (backend->Init())
+      LOG(INFO) << "Using GNOME keyring for password storage.";
+    else
+      backend.reset();
+  }
+  // TODO(mdm): this can change to a WARNING when we detect by default.
+  if (!backend.get())
+    LOG(INFO) << "Using default (unencrypted) store for password storage.";
+
+  ps = new PasswordStoreX(login_db, this,
+                          GetWebDataService(Profile::IMPLICIT_ACCESS),
+                          backend.release());
 #else
   NOTIMPLEMENTED();
 #endif
   if (!ps)
     delete login_db;
 
   if (!ps || !ps->Init()) {
-    NOTREACHED() << "Could not initialise password manager";
+    NOTREACHED() << "Could not initialize password manager.";
     return;
   }
   password_store_.swap(ps);
 }
 
 DownloadManager* ProfileImpl::GetDownloadManager() {
   if (!created_download_manager_) {
     scoped_refptr<DownloadManager> dlm(new DownloadManager);
     dlm->Init(this);
     created_download_manager_ = true;
@@ skipping 254 matching lines @@
                                  CommandLine::ForCurrentProcess()));
   sync_service_.reset(
       profile_sync_factory_->CreateProfileSyncService());
   sync_service_->Initialize();
 }
 
 void ProfileImpl::InitCloudPrintProxyService() {
   cloud_print_proxy_service_.reset(new CloudPrintProxyService(this));
   cloud_print_proxy_service_->Initialize();
 }