Remove the test-only X509Certificate constructor

chrome/browser/chromeos/login/users/multi_profile_user_controller_unittest.cc

Index: chrome/browser/chromeos/login/users/multi_profile_user_controller_unittest.cc
diff --git a/chrome/browser/chromeos/login/users/multi_profile_user_controller_unittest.cc b/chrome/browser/chromeos/login/users/multi_profile_user_controller_unittest.cc
index b32eb2cf2f2b09c533b2bfa590ca09805ac6c8f6..ddb1fbd74458ff17d4d6441844fd20b8ab53b297 100644
--- a/chrome/browser/chromeos/login/users/multi_profile_user_controller_unittest.cc
+++ b/chrome/browser/chromeos/login/users/multi_profile_user_controller_unittest.cc
@@ -27,7 +27,9 @@
 #include "components/syncable_prefs/testing_pref_service_syncable.h"
 #include "components/user_manager/user_manager.h"
 #include "content/public/test/test_browser_thread_bundle.h"
+#include "net/base/test_data_directory.h"
 #include "net/cert/x509_certificate.h"
+#include "net/test/cert_test_util.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace chromeos {
@@ -439,8 +441,8 @@ TEST_F(MultiProfileUserControllerTest,
             MultiProfileUserController::GetPrimaryUserPolicy());
 
   net::CertificateList certificates;
-  certificates.push_back(new net::X509Certificate(
-      "subject", "issuer", base::Time(), base::Time()));
+  certificates.push_back(
+      net::ImportCertFromFile(net::GetTestCertsDirectory(), "ok_cert.pem"));

[Nico, 2016/06/22 14:12:11]

(is there a bug somewhere for making CertificateList have unique_ptrs instead of
raw pointers?)

[Ryan Sleevi, 2016/06/22 15:22:26]

CertificateList takes scoped_refptr<>s

   service->OnTrustAnchorsChanged(certificates);
   EXPECT_TRUE(service->has_policy_certificates());
   EXPECT_FALSE(controller()->IsUserAllowedInSession(