Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1716)

Unified Diff: chrome/browser/extensions/api/enterprise_platform_keys/enterprise_platform_keys_apitest_nss.cc

Issue 325313004: Cleanup tests using SetSkipCertificateImporterCreationForTest (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/enterprise_platform_keys/enterprise_platform_keys_apitest_nss.cc
diff --git a/chrome/browser/extensions/api/enterprise_platform_keys/enterprise_platform_keys_apitest_nss.cc b/chrome/browser/extensions/api/enterprise_platform_keys/enterprise_platform_keys_apitest_nss.cc
index 5d8c59aff53cdb0fe5ea96417e82653275a800be..825b06a209b5681fb9ce7c4b9237ac012b81735d 100644
--- a/chrome/browser/extensions/api/enterprise_platform_keys/enterprise_platform_keys_apitest_nss.cc
+++ b/chrome/browser/extensions/api/enterprise_platform_keys/enterprise_platform_keys_apitest_nss.cc
@@ -101,12 +101,6 @@ void DidGetCertDatabase(base::RunLoop* loop, net::NSSCertDatabase* cert_db) {
class EnterprisePlatformKeysTest : public ExtensionApiTest {
public:
- virtual void SetUp() {
- policy::UserNetworkConfigurationUpdater::
- SetSkipCertificateImporterCreationForTest(true /*skip*/);
- ExtensionApiTest::SetUp();
- }
-
virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
ExtensionApiTest::SetUpCommandLine(command_line);
@@ -135,16 +129,6 @@ class EnterprisePlatformKeysTest : public ExtensionApiTest {
FROM_HERE,
base::Bind(chrome_browser_net::SetUrlRequestMocksEnabled, true));
- {
- base::RunLoop loop;
- content::BrowserThread::PostTask(
- content::BrowserThread::IO,
- FROM_HERE,
- base::Bind(&EnterprisePlatformKeysTest::SetupTestNSSDBOnIOThread,
- base::Unretained(this),
- &loop));
- loop.Run();
- }
{
base::RunLoop loop;
@@ -156,31 +140,7 @@ class EnterprisePlatformKeysTest : public ExtensionApiTest {
SetPolicy();
}
- virtual void CleanUpOnMainThread() OVERRIDE {
- base::RunLoop loop;
- content::BrowserThread::PostTask(
- content::BrowserThread::IO,
- FROM_HERE,
- base::Bind(&EnterprisePlatformKeysTest::DeleteTestNSSDBOnIOThread,
- base::Unretained(this),
- &loop));
- loop.Run();
- ExtensionApiTest::CleanUpOnMainThread();
- }
-
private:
- void SetupTestNSSDBOnIOThread(base::RunLoop* loop) {
- test_nssdb_.reset(new crypto::ScopedTestNSSDB);
- content::BrowserThread::PostTask(
- content::BrowserThread::UI, FROM_HERE, loop->QuitClosure());
- }
-
- void DeleteTestNSSDBOnIOThread(base::RunLoop* loop) {
- test_nssdb_.reset();
- content::BrowserThread::PostTask(
- content::BrowserThread::UI, FROM_HERE, loop->QuitClosure());
- }
-
void SetPolicy() {
// Extensions that are force-installed come from an update URL, which
// defaults to the webstore. Use a mock URL for this test with an update
@@ -209,7 +169,6 @@ class EnterprisePlatformKeysTest : public ExtensionApiTest {
observer.Wait();
}
- scoped_ptr<crypto::ScopedTestNSSDB> test_nssdb_;
policy::MockConfigurationPolicyProvider policy_provider_;
};

Powered by Google App Engine
This is Rietveld 408576698