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

Unified Diff: content/browser/media/webrtc_identity_store_unittest.cc

Issue 15969025: Generates the DTLS identity in browser process and returns it to render process. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 6 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: content/browser/media/webrtc_identity_store_unittest.cc
diff --git a/content/browser/media/webrtc_identity_store_unittest.cc b/content/browser/media/webrtc_identity_store_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..bf2d0079f16189fdcab4736fb6c5d0f6f164fdc7
--- /dev/null
+++ b/content/browser/media/webrtc_identity_store_unittest.cc
@@ -0,0 +1,107 @@
+// Copyright (c) 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 "base/bind.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/message_loop.h"
+#include "base/run_loop.h"
+#include "base/synchronization/waitable_event.h"
+#include "base/threading/sequenced_worker_pool.h"
+#include "content/browser/media/webrtc_identity_store.h"
+#include "content/browser/renderer_host/media/webrtc_identity_service_host.h"
+#include "content/common/media/webrtc_identity_messages.h"
+#include "content/public/test/test_browser_thread_bundle.h"
+#include "content/public/test/test_utils.h"
+#include "googleurl/src/gurl.h"
+#include "net/base/net_errors.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace content {
+
+class WebRTCIdentityStoreTest : public testing::Test {
+ public:
+ WebRTCIdentityStoreTest()
+ : browser_thread_bundle_(TestBrowserThreadBundle::IO_MAINLOOP),
+ sequenced_worker_pool_(
+ new base::SequencedWorkerPool(3, "ServerBoundCertServiceTest")) {
Ami GONE FROM CHROMIUM 2013/06/27 20:05:18 "ServerBoundCertServiceTest" is copy/pasta?
jiayl 2013/06/27 21:08:37 Done.
+ webrtc_identity_store_.reset(
+ new WebRTCIdentityStore(sequenced_worker_pool_));
+ }
+
+ virtual ~WebRTCIdentityStoreTest() { sequenced_worker_pool_->Shutdown(); }
+
+ protected:
+ TestBrowserThreadBundle browser_thread_bundle_;
+ scoped_refptr<base::SequencedWorkerPool> sequenced_worker_pool_;
+ scoped_ptr<WebRTCIdentityStore> webrtc_identity_store_;
+};
+
+namespace {
Ami GONE FROM CHROMIUM 2013/06/27 20:05:18 nit: prefer to make ORC below static and the TEST_
jiayl 2013/06/27 21:08:37 Done.
+
+void OnRequestCompleted(bool* completed,
+ int error,
+ const std::string& certificate,
+ const std::string& private_key) {
+ ASSERT_EQ(net::OK, error);
+ ASSERT_NE("", certificate);
+ ASSERT_NE("", private_key);
+ *completed = true;
+}
+
+TEST_F(WebRTCIdentityStoreTest, RequestIdentity) {
+ scoped_ptr<bool> completed(new bool(false));
Ami GONE FROM CHROMIUM 2013/06/27 20:05:18 why not just bool completed = false; and pass &com
jiayl 2013/06/27 21:08:37 Done.
+ base::Closure cancel_callback;
+ webrtc_identity_store_->RequestIdentity(
Ami GONE FROM CHROMIUM 2013/06/27 20:05:18 ASSERT_TRUE?
jiayl 2013/06/27 21:08:37 Done.
+ GURL("http://google.com"),
+ "a",
+ "b",
+ base::Bind(&OnRequestCompleted, base::Unretained(completed.get())),
+ &cancel_callback);
+ sequenced_worker_pool_->FlushForTesting();
+ base::RunLoop().RunUntilIdle();
+ EXPECT_EQ(true, *completed);
+}
+
+TEST_F(WebRTCIdentityStoreTest, CancelRequest) {
+ scoped_ptr<bool> completed(new bool(false));
+ base::Closure cancel_callback;
+ bool success = webrtc_identity_store_->RequestIdentity(
+ GURL("http://google.com"),
+ "a",
+ "b",
+ base::Bind(&OnRequestCompleted, base::Unretained(completed.get())),
+ &cancel_callback);
+ EXPECT_TRUE(success);
+ cancel_callback.Run();
+ sequenced_worker_pool_->FlushForTesting();
+ base::RunLoop().RunUntilIdle();
+ EXPECT_EQ(false, *completed);
+}
+
+TEST_F(WebRTCIdentityStoreTest, MultipleRequests) {
+ scoped_ptr<bool> completed_1(new bool(false));
+ scoped_ptr<bool> completed_2(new bool(false));
+ base::Closure cancel_callback;
+ webrtc_identity_store_->RequestIdentity(
Ami GONE FROM CHROMIUM 2013/06/27 20:05:18 ditto ASSERT_TRUE
jiayl 2013/06/27 21:08:37 Done.
+ GURL("http://foo.com"),
+ "a",
+ "b",
+ base::Bind(&OnRequestCompleted, base::Unretained(completed_1.get())),
+ &cancel_callback);
+
+ webrtc_identity_store_->RequestIdentity(
Ami GONE FROM CHROMIUM 2013/06/27 20:05:18 ditto ASSERT_TRUE
jiayl 2013/06/27 21:08:37 Done.
+ GURL("http://bar.com"),
+ "a",
+ "b",
+ base::Bind(&OnRequestCompleted, base::Unretained(completed_2.get())),
+ &cancel_callback);
+
+ sequenced_worker_pool_->FlushForTesting();
+ base::RunLoop().RunUntilIdle();
+ EXPECT_EQ(true, *completed_1);
+ EXPECT_EQ(true, *completed_2);
+}
+
+} // namespace
+} // namespace content

Powered by Google App Engine
This is Rietveld 408576698