NSSCertDatabaseChromeOS

chrome/browser/net/nss_context_chromeos_browsertest.cc

Index: chrome/browser/net/nss_context_chromeos_browsertest.cc
diff --git a/chrome/browser/net/nss_context_chromeos_browsertest.cc b/chrome/browser/net/nss_context_chromeos_browsertest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..3f4b6591ea602aaa452792542ea6b159444220a5
--- /dev/null
+++ b/chrome/browser/net/nss_context_chromeos_browsertest.cc
@@ -0,0 +1,162 @@
+// Copyright 2013 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/net/nss_context.h"
+
+#include "base/bind.h"
+#include "base/run_loop.h"
+#include "chrome/browser/chromeos/login/login_manager_test.h"
+#include "chrome/browser/chromeos/login/startup_utils.h"
+#include "chrome/browser/chromeos/login/user.h"
+#include "chrome/browser/chromeos/login/user_adding_screen.h"
+#include "chrome/browser/chromeos/login/user_manager.h"
+#include "content/public/browser/browser_thread.h"
+#include "net/cert/nss_cert_database.h"
+
+namespace {
+
+const char kTestUser1[] = "test-user1@gmail.com";
+const char kTestUser2[] = "test-user2@gmail.com";
+
+void NotCalledDbCallback(net::NSSCertDatabase* db) { ASSERT_TRUE(false); }
+
+// DBTester handles retrieving the NSSCertDatabase for a given profile, and
+// doing some simple sanity checks.
+// Browser test cases run on the UI thread, while the nss_context access needs
+// to happen on the IO thread. The DBTester class encapsulates the thread
+// posting and waiting on the UI thread so that the test case body can be
+// written linearly.
+class DBTester {
+ public:
+  explicit DBTester(Profile* profile) : profile_(profile), db_(NULL) {}
+
+  // Initial retrieval of cert database. It may be asynchronous or synchronous.
+  // Returns true if the database was retrieved successfully.
+  bool DoGetDBTests() {
+    base::RunLoop run_loop;
+    content::BrowserThread::PostTask(
+        content::BrowserThread::IO,
+        FROM_HERE,
+        base::Bind(&DBTester::GetDBAndDoTestsOnIOThread,
+                   base::Unretained(this),
+                   profile_->GetResourceContext(),
+                   run_loop.QuitClosure()));
+    run_loop.Run();
+    return !!db_;
+  }
+
+  // Test retrieving the database again, should be called after DoGetDBTests.
+  void DoGetDBAgainTests() {
+    base::RunLoop run_loop;
+    content::BrowserThread::PostTask(
+        content::BrowserThread::IO,
+        FROM_HERE,
+        base::Bind(&DBTester::DoGetDBAgainTestsOnIOThread,
+                   base::Unretained(this),
+                   profile_->GetResourceContext(),
+                   run_loop.QuitClosure()));
+    run_loop.Run();
+  }
+
+  void DoNotEqualsTests(DBTester* other_tester) {
+    // The DB and its NSS slots should be different for each profile.
+    EXPECT_NE(db_, other_tester->db_);
+    EXPECT_NE(db_->GetPublicSlot().get(),
+              other_tester->db_->GetPublicSlot().get());
+  }
+
+ private:
+  void GetDBAndDoTestsOnIOThread(content::ResourceContext* context,
+                                 const base::Closure& done_callback) {
+    net::NSSCertDatabase* db = GetNSSCertDatabaseForResourceContext(
+        context,
+        base::Bind(&DBTester::DoTestsOnIOThread,
+                   base::Unretained(this),
+                   done_callback));
+    if (db) {
+      DVLOG(1) << "got db synchronously";
+      DoTestsOnIOThread(done_callback, db);
+    } else {
+      DVLOG(1) << "getting db asynchronously...";
+    }
+  }
+
+  void DoTestsOnIOThread(const base::Closure& done_callback,
+                         net::NSSCertDatabase* db) {
+    db_ = db;
+    EXPECT_TRUE(db);
+    if (db) {
+      EXPECT_TRUE(db->GetPublicSlot().get());
+      // Public and private slot are the same in tests.
+      EXPECT_EQ(db->GetPublicSlot().get(), db->GetPrivateSlot().get());
+    }
+
+    content::BrowserThread::PostTask(
+        content::BrowserThread::UI, FROM_HERE, done_callback);
+  }
+
+  void DoGetDBAgainTestsOnIOThread(content::ResourceContext* context,
+                                   const base::Closure& done_callback) {
+    net::NSSCertDatabase* db = GetNSSCertDatabaseForResourceContext(
+        context, base::Bind(&NotCalledDbCallback));
+    // Should always be synchronous now.
+    EXPECT_TRUE(db);
+    // Should return the same db as before.
+    EXPECT_EQ(db_, db);
+
+    content::BrowserThread::PostTask(
+        content::BrowserThread::UI, FROM_HERE, done_callback);
+  }
+
+  Profile* profile_;
+  net::NSSCertDatabase* db_;
+};
+
+}  // namespace
+
+class NSSContextChromeOSBrowserTest : public chromeos::LoginManagerTest {
+ public:
+  NSSContextChromeOSBrowserTest()
+      : LoginManagerTest(true /* should_launch_browser */) {}
+  virtual ~NSSContextChromeOSBrowserTest() {}
+};
+
+IN_PROC_BROWSER_TEST_F(NSSContextChromeOSBrowserTest, PRE_TwoUsers) {
+  // Initialization for ChromeOS multi-profile test infrastructure.
+  RegisterUser(kTestUser1);
+  RegisterUser(kTestUser2);
+  chromeos::StartupUtils::MarkOobeCompleted();
+}
+
+IN_PROC_BROWSER_TEST_F(NSSContextChromeOSBrowserTest, TwoUsers) {
+  chromeos::UserManager* user_manager = chromeos::UserManager::Get();
+
+  // Log in first user and get their DB.
+  LoginUser(kTestUser1);
+  Profile* profile1 =
+      user_manager->GetProfileByUser(user_manager->FindUser(kTestUser1));
+  ASSERT_TRUE(profile1);
+
+  DBTester tester1(profile1);
+  ASSERT_TRUE(tester1.DoGetDBTests());
+
+  // Log in second user and get their DB.
+  chromeos::UserAddingScreen::Get()->Start();
+  base::RunLoop().RunUntilIdle();
+  AddUser(kTestUser2);
+
+  Profile* profile2 =
+      user_manager->GetProfileByUser(user_manager->FindUser(kTestUser2));
+  ASSERT_TRUE(profile2);
+
+  DBTester tester2(profile2);
+  ASSERT_TRUE(tester2.DoGetDBTests());
+
+  // Get both DBs again to check that the same object is returned.
+  tester1.DoGetDBAgainTests();
+  tester2.DoGetDBAgainTests();
+
+  // Check that each user has a separate DB and NSS slots.
+  tester1.DoNotEqualsTests(&tester2);
+}