Decode h264 fmtp sprop-parameter-sets to binary.

webrtc/modules/video_coding/h264_sprop_parameter_sets_unittest.cc

Index: webrtc/modules/video_coding/h264_sprop_parameter_sets_unittest.cc
diff --git a/webrtc/modules/video_coding/h264_sprop_parameter_sets_unittest.cc b/webrtc/modules/video_coding/h264_sprop_parameter_sets_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..8ade3774efdb2c76a72e37cd5bbc320efaf9d4b4
--- /dev/null
+++ b/webrtc/modules/video_coding/h264_sprop_parameter_sets_unittest.cc
@@ -0,0 +1,43 @@
+/*
+ *  Copyright (c) 2016 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 "webrtc/modules/video_coding/h264_sprop_parameter_sets.h"
+
+#include "webrtc/base/arraysize.h"
+#include "webrtc/base/bitbuffer.h"

[sprang_webrtc, 2016/12/06 18:23:18]

Are these used?

[johan, 2016/12/07 17:10:40]

Removing them and adding actual used headers.

+#include "webrtc/test/gtest.h"
+
+namespace webrtc {
+
+class H264SpropParameterSetsTest : public testing::Test {
+ public:
+  H264SpropParameterSets h264_sprop;
+};
+
+TEST_F(H264SpropParameterSetsTest, Base64DecodeSprop) {
+  // Example sprop string from https://tools.ietf.org/html/rfc3984 .
+  EXPECT_TRUE(h264_sprop.DecodeSprop("Z0IACpZTBYmI,aMljiA=="));
+  static const std::vector<uint8_t> raw_sps{0x67, 0x42, 0x00, 0x0A, 0x96,
+                                            0x53, 0x05, 0x89, 0x88};
+  static const std::vector<uint8_t> raw_pps{0x68, 0xC9, 0x63, 0x88};
+  EXPECT_EQ(raw_sps, h264_sprop.sps_nalu());
+  EXPECT_EQ(raw_pps, h264_sprop.pps_nalu());
+}
+
+TEST_F(H264SpropParameterSetsTest, InvalidData) {

[sprang_webrtc, 2016/12/06 18:23:18]

maybe add a non-empty string without comma and one with two commas - just for
completeness?

[johan, 2016/12/07 17:10:40]

Good one.

+  EXPECT_FALSE(h264_sprop.DecodeSprop(","));
+  EXPECT_FALSE(h264_sprop.DecodeSprop(""));
+  EXPECT_FALSE(h264_sprop.DecodeSprop(",iA=="));
+  EXPECT_FALSE(h264_sprop.DecodeSprop("iA==,"));
+  EXPECT_TRUE(h264_sprop.DecodeSprop("iA==,iA=="));
+  EXPECT_FALSE(h264_sprop.DecodeSprop("--,--"));
+}
+
+}  // namespace webrtc