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..4ff5086f3f43b8fe52bcee652c15e7de9f6505e4
--- /dev/null
+++ b/chrome/browser/net/nss_context_chromeos_browsertest.cc
@@ -0,0 +1,146 @@
+// 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); }
+
+class DBTester {

[Ryan Sleevi, 2013/12/18 21:28:32]

Please add comments throughout here to document, even if it's as simple as
explaining why you're using this helper DBTester class

[mattm, 2013/12/19 22:35:00]

Done.

+ public:
+  explicit DBTester(Profile* profile) : profile_(profile), db_(NULL) {}
+
+  void 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();
+  }
+
+  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) {}
+  virtual ~NSSContextChromeOSBrowserTest() {}
+};
+
+IN_PROC_BROWSER_TEST_F(NSSContextChromeOSBrowserTest, PRE_TwoUsers) {
+  RegisterUser(kTestUser1);
+  RegisterUser(kTestUser2);
+  chromeos::StartupUtils::MarkOobeCompleted();
+}
+
+IN_PROC_BROWSER_TEST_F(NSSContextChromeOSBrowserTest, TwoUsers) {
+  chromeos::UserManager* user_manager = chromeos::UserManager::Get();
+
+  LoginUser(kTestUser1);
+  Profile* profile1 =
+      user_manager->GetProfileByUser(user_manager->FindUser(kTestUser1));
+  ASSERT_TRUE(profile1);
+
+  DBTester tester1(profile1);
+  tester1.DoGetDBTests();
+
+  chromeos::UserAddingScreen::Get()->Start();
+  base::RunLoop().RunUntilIdle();
+  AddUser(kTestUser2);
+
+  Profile* profile2 =
+      user_manager->GetProfileByUser(user_manager->FindUser(kTestUser2));
+  ASSERT_TRUE(profile2);
+
+  DBTester tester2(profile2);
+  tester2.DoGetDBTests();
+
+  tester1.DoGetDBAgainTests();
+  tester2.DoGetDBAgainTests();
+
+  tester1.DoNotEqualsTests(&tester2);
+}