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

Unified Diff: chrome/browser/browsing_data/browsing_data_server_bound_cert_helper_unittest.cc

Issue 356713005: Rename ServerBoundCert => ChannelID to reflect the current name (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix cookies_list.js 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/browsing_data/browsing_data_server_bound_cert_helper_unittest.cc
diff --git a/chrome/browser/browsing_data/browsing_data_server_bound_cert_helper_unittest.cc b/chrome/browser/browsing_data/browsing_data_server_bound_cert_helper_unittest.cc
deleted file mode 100644
index a95405e3627d7ca169c23356eec679961fb07a09..0000000000000000000000000000000000000000
--- a/chrome/browser/browsing_data/browsing_data_server_bound_cert_helper_unittest.cc
+++ /dev/null
@@ -1,144 +0,0 @@
-// Copyright (c) 2012 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/browsing_data/browsing_data_server_bound_cert_helper.h"
-
-#include "base/bind.h"
-#include "base/run_loop.h"
-#include "chrome/test/base/testing_profile.h"
-#include "content/public/browser/browser_thread.h"
-#include "content/public/test/test_browser_thread_bundle.h"
-#include "net/ssl/server_bound_cert_service.h"
-#include "net/url_request/url_request_context.h"
-#include "net/url_request/url_request_context_getter.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-using content::BrowserThread;
-
-class BrowsingDataServerBoundCertHelperTest
- : public testing::Test,
- public net::SSLConfigService::Observer {
- public:
- BrowsingDataServerBoundCertHelperTest() : ssl_config_changed_count_(0) {
- }
-
- virtual void SetUp() OVERRIDE {
- testing_profile_.reset(new TestingProfile());
-
- testing_profile_->GetSSLConfigService()->AddObserver(this);
- }
-
- virtual void TearDown() OVERRIDE {
- testing_profile_->GetSSLConfigService()->RemoveObserver(this);
- }
-
- void CreateCertsForTest() {
- net::URLRequestContext* context =
- testing_profile_->GetRequestContext()->GetURLRequestContext();
- net::ServerBoundCertStore* cert_store =
- context->server_bound_cert_service()->GetCertStore();
- cert_store->SetServerBoundCert("https://www.google.com:443",
- base::Time(), base::Time(),
- "key", "cert");
- cert_store->SetServerBoundCert("https://www.youtube.com:443",
- base::Time(), base::Time(),
- "key", "cert");
- }
-
- void FetchCallback(
- const net::ServerBoundCertStore::ServerBoundCertList& certs) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- server_bound_cert_list_ = certs;
- }
-
- // net::SSLConfigService::Observer implementation:
- virtual void OnSSLConfigChanged() OVERRIDE {
- ssl_config_changed_count_++;
- }
-
- protected:
- content::TestBrowserThreadBundle thread_bundle_;
- scoped_ptr<TestingProfile> testing_profile_;
-
- net::ServerBoundCertStore::ServerBoundCertList server_bound_cert_list_;
-
- int ssl_config_changed_count_;
-};
-
-TEST_F(BrowsingDataServerBoundCertHelperTest, FetchData) {
- CreateCertsForTest();
- scoped_refptr<BrowsingDataServerBoundCertHelper> helper(
- BrowsingDataServerBoundCertHelper::Create(testing_profile_.get()));
-
- helper->StartFetching(
- base::Bind(&BrowsingDataServerBoundCertHelperTest::FetchCallback,
- base::Unretained(this)));
-
- // Blocks until BrowsingDataServerBoundCertHelperTest::FetchCallback is
- // notified.
- base::RunLoop().RunUntilIdle();
-
- ASSERT_EQ(2UL, server_bound_cert_list_.size());
- net::ServerBoundCertStore::ServerBoundCertList::const_iterator it =
- server_bound_cert_list_.begin();
-
- // Correct because fetching server_bound_cert_list_ will get them out in the
- // same order CreateCertsForTest put them in.
- ASSERT_TRUE(it != server_bound_cert_list_.end());
- EXPECT_EQ("https://www.google.com:443", it->server_identifier());
-
- ASSERT_TRUE(++it != server_bound_cert_list_.end());
- EXPECT_EQ("https://www.youtube.com:443", it->server_identifier());
-
- ASSERT_TRUE(++it == server_bound_cert_list_.end());
-
- EXPECT_EQ(0, ssl_config_changed_count_);
-}
-
-TEST_F(BrowsingDataServerBoundCertHelperTest, DeleteCert) {
- CreateCertsForTest();
- scoped_refptr<BrowsingDataServerBoundCertHelper> helper(
- BrowsingDataServerBoundCertHelper::Create(testing_profile_.get()));
-
- helper->DeleteServerBoundCert("https://www.google.com:443");
-
- helper->StartFetching(
- base::Bind(&BrowsingDataServerBoundCertHelperTest::FetchCallback,
- base::Unretained(this)));
- base::RunLoop().RunUntilIdle();
-
- EXPECT_EQ(1, ssl_config_changed_count_);
- ASSERT_EQ(1UL, server_bound_cert_list_.size());
- net::ServerBoundCertStore::ServerBoundCertList::const_iterator it =
- server_bound_cert_list_.begin();
-
- ASSERT_TRUE(it != server_bound_cert_list_.end());
- EXPECT_EQ("https://www.youtube.com:443", it->server_identifier());
-
- ASSERT_TRUE(++it == server_bound_cert_list_.end());
-
- helper->DeleteServerBoundCert("https://www.youtube.com:443");
-
- helper->StartFetching(
- base::Bind(&BrowsingDataServerBoundCertHelperTest::FetchCallback,
- base::Unretained(this)));
- base::RunLoop().RunUntilIdle();
-
- EXPECT_EQ(2, ssl_config_changed_count_);
- ASSERT_EQ(0UL, server_bound_cert_list_.size());
-}
-
-TEST_F(BrowsingDataServerBoundCertHelperTest, CannedEmpty) {
- std::string origin = "https://www.google.com";
-
- scoped_refptr<CannedBrowsingDataServerBoundCertHelper> helper(
- new CannedBrowsingDataServerBoundCertHelper());
-
- ASSERT_TRUE(helper->empty());
- helper->AddServerBoundCert(net::ServerBoundCertStore::ServerBoundCert(
- origin, base::Time(), base::Time(), "key", "cert"));
- ASSERT_FALSE(helper->empty());
- helper->Reset();
- ASSERT_TRUE(helper->empty());
-}

Powered by Google App Engine
This is Rietveld 408576698