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

Unified Diff: webrtc/media/webrtc/simulcast_unittest.cc

Issue 1587193006: Move talk/media to webrtc/media (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Disable sign-compare warning on Win Clang Created 4 years, 11 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: webrtc/media/webrtc/simulcast_unittest.cc
diff --git a/webrtc/media/webrtc/simulcast_unittest.cc b/webrtc/media/webrtc/simulcast_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..7ea37c0091968b055fed1d568dd3e54ee880b93f
--- /dev/null
+++ b/webrtc/media/webrtc/simulcast_unittest.cc
@@ -0,0 +1,56 @@
+/*
+ * Copyright 2014 The WebRTC Project Authors. All rights reserved.
+ *
+ * Use of this source code is governed by a BSD-style license
+ * that can be found in the LICENSE file in the root of the source
+ * tree. An additional intellectual property rights grant can be found
+ * in the file PATENTS. All contributing project authors may
+ * be found in the AUTHORS file in the root of the source tree.
+ */
+
+#include <string>
+
+#include "testing/gtest/include/gtest/gtest.h"
+#include "webrtc/media/webrtc/simulcast.h"
+
+namespace cricket {
+
+class ScreenshareLayerConfigTest : public testing::Test,
+ protected ScreenshareLayerConfig {
+ public:
+ ScreenshareLayerConfigTest() : ScreenshareLayerConfig(0, 0) {}
+
+ void ExpectParsingFails(const std::string& group) {
+ ScreenshareLayerConfig config(100, 1000);
+ EXPECT_FALSE(FromFieldTrialGroup(group, &config));
+ }
+};
+
+TEST_F(ScreenshareLayerConfigTest, UsesDefaultBitrateConfigForDefaultGroup) {
+ ExpectParsingFails("");
+}
+
+TEST_F(ScreenshareLayerConfigTest, UsesDefaultConfigForInvalidBitrates) {
+ ExpectParsingFails("-");
+ ExpectParsingFails("1-");
+ ExpectParsingFails("-1");
+ ExpectParsingFails("-12");
+ ExpectParsingFails("12-");
+ ExpectParsingFails("booh!");
+ ExpectParsingFails("1-b");
+ ExpectParsingFails("a-2");
+ ExpectParsingFails("49-1000");
+ ExpectParsingFails("50-6001");
+ ExpectParsingFails("100-99");
+ ExpectParsingFails("1002003004005006-99");
+ ExpectParsingFails("99-1002003004005006");
+}
+
+TEST_F(ScreenshareLayerConfigTest, ParsesValidBitrateConfig) {
+ ScreenshareLayerConfig config(100, 1000);
+ EXPECT_TRUE(ScreenshareLayerConfig::FromFieldTrialGroup("101-1001", &config));
+ EXPECT_EQ(101, config.tl0_bitrate_kbps);
+ EXPECT_EQ(1001, config.tl1_bitrate_kbps);
+}
+
+} // namespace cricket

Powered by Google App Engine
This is Rietveld 408576698