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

Unified Diff: media/formats/mp4/avc_unittest.cc

Issue 1534273002: Switch to standard integer types in media/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: more Created 5 years 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: media/formats/mp4/avc_unittest.cc
diff --git a/media/formats/mp4/avc_unittest.cc b/media/formats/mp4/avc_unittest.cc
index 19f10f3a602c1b7b9b4be5e5ec122dc8188633c3..2faf56e7d8526e2c899382ce505fb660a699ec35 100644
--- a/media/formats/mp4/avc_unittest.cc
+++ b/media/formats/mp4/avc_unittest.cc
@@ -4,7 +4,6 @@
#include <string.h>
-#include "base/basictypes.h"
#include "base/strings/string_split.h"
#include "base/strings/string_util.h"
#include "media/base/decrypt_config.h"
@@ -17,16 +16,15 @@
namespace media {
namespace mp4 {
-static const uint8 kNALU1[] = { 0x01, 0x02, 0x03 };
-static const uint8 kNALU2[] = { 0x04, 0x05, 0x06, 0x07 };
-static const uint8 kExpected[] = {
- 0x00, 0x00, 0x00, 0x01, 0x01, 0x02, 0x03,
- 0x00, 0x00, 0x00, 0x01, 0x04, 0x05, 0x06, 0x07 };
+static const uint8_t kNALU1[] = {0x01, 0x02, 0x03};
+static const uint8_t kNALU2[] = {0x04, 0x05, 0x06, 0x07};
+static const uint8_t kExpected[] = {0x00, 0x00, 0x00, 0x01, 0x01,
+ 0x02, 0x03, 0x00, 0x00, 0x00,
+ 0x01, 0x04, 0x05, 0x06, 0x07};
-static const uint8 kExpectedParamSets[] = {
- 0x00, 0x00, 0x00, 0x01, 0x67, 0x12,
- 0x00, 0x00, 0x00, 0x01, 0x67, 0x34,
- 0x00, 0x00, 0x00, 0x01, 0x68, 0x56, 0x78};
+static const uint8_t kExpectedParamSets[] = {
+ 0x00, 0x00, 0x00, 0x01, 0x67, 0x12, 0x00, 0x00, 0x00, 0x01,
+ 0x67, 0x34, 0x00, 0x00, 0x00, 0x01, 0x68, 0x56, 0x78};
static H264NALU::Type StringToNALUType(const std::string& name) {
if (name == "P")
@@ -111,7 +109,7 @@ static std::string NALUTypeToString(int type) {
return "UnsupportedType";
}
-static void WriteStartCodeAndNALUType(std::vector<uint8>* buffer,
+static void WriteStartCodeAndNALUType(std::vector<uint8_t>* buffer,
const std::string& nal_unit_type) {
buffer->push_back(0x00);
buffer->push_back(0x00);
@@ -131,7 +129,8 @@ static void WriteStartCodeAndNALUType(std::vector<uint8>* buffer,
// The output buffer will contain a valid-looking Annex B (it's valid-looking in
// the sense that it has start codes and correct NALU types, but the actual NALU
// payload is junk).
-void StringToAnnexB(const std::string& str, std::vector<uint8>* buffer,
+void StringToAnnexB(const std::string& str,
+ std::vector<uint8_t>* buffer,
std::vector<SubsampleEntry>* subsamples) {
DCHECK(!str.empty());
@@ -174,7 +173,7 @@ void StringToAnnexB(const std::string& str, std::vector<uint8>* buffer,
}
}
-std::string AnnexBToString(const std::vector<uint8>& buffer,
+std::string AnnexBToString(const std::vector<uint8_t>& buffer,
const std::vector<SubsampleEntry>& subsamples) {
std::stringstream ss;
@@ -202,7 +201,7 @@ std::string AnnexBToString(const std::vector<uint8>& buffer,
class AVCConversionTest : public testing::TestWithParam<int> {
protected:
- void WriteLength(int length_size, int length, std::vector<uint8>* buf) {
+ void WriteLength(int length_size, int length, std::vector<uint8_t>* buf) {
DCHECK_GE(length, 0);
DCHECK_LE(length, 255);
@@ -211,7 +210,7 @@ class AVCConversionTest : public testing::TestWithParam<int> {
buf->push_back(length);
}
- void MakeInputForLength(int length_size, std::vector<uint8>* buf) {
+ void MakeInputForLength(int length_size, std::vector<uint8_t>* buf) {
buf->clear();
WriteLength(length_size, sizeof(kNALU1), buf);
@@ -224,7 +223,7 @@ class AVCConversionTest : public testing::TestWithParam<int> {
};
TEST_P(AVCConversionTest, ParseCorrectly) {
- std::vector<uint8> buf;
+ std::vector<uint8_t> buf;
std::vector<SubsampleEntry> subsamples;
MakeInputForLength(GetParam(), &buf);
EXPECT_TRUE(AVC::ConvertFrameToAnnexB(GetParam(), &buf, &subsamples));
@@ -236,14 +235,14 @@ TEST_P(AVCConversionTest, ParseCorrectly) {
// Intentionally write NALU sizes that are larger than the buffer.
TEST_P(AVCConversionTest, NALUSizeTooLarge) {
- std::vector<uint8> buf;
+ std::vector<uint8_t> buf;
WriteLength(GetParam(), 10 * sizeof(kNALU1), &buf);
buf.insert(buf.end(), kNALU1, kNALU1 + sizeof(kNALU1));
EXPECT_FALSE(AVC::ConvertFrameToAnnexB(GetParam(), &buf, nullptr));
}
TEST_P(AVCConversionTest, NALUSizeIsZero) {
- std::vector<uint8> buf;
+ std::vector<uint8_t> buf;
WriteLength(GetParam(), 0, &buf);
WriteLength(GetParam(), sizeof(kNALU1), &buf);
@@ -258,7 +257,7 @@ TEST_P(AVCConversionTest, NALUSizeIsZero) {
}
TEST_P(AVCConversionTest, SubsampleSizesUpdatedAfterAnnexBConversion) {
- std::vector<uint8> buf;
+ std::vector<uint8_t> buf;
std::vector<SubsampleEntry> subsamples;
SubsampleEntry subsample;
@@ -298,7 +297,7 @@ TEST_P(AVCConversionTest, SubsampleSizesUpdatedAfterAnnexBConversion) {
}
TEST_P(AVCConversionTest, ParsePartial) {
- std::vector<uint8> buf;
+ std::vector<uint8_t> buf;
MakeInputForLength(GetParam(), &buf);
buf.pop_back();
EXPECT_FALSE(AVC::ConvertFrameToAnnexB(GetParam(), &buf, nullptr));
@@ -312,7 +311,7 @@ TEST_P(AVCConversionTest, ParsePartial) {
}
TEST_P(AVCConversionTest, ParseEmpty) {
- std::vector<uint8> buf;
+ std::vector<uint8_t> buf;
EXPECT_TRUE(AVC::ConvertFrameToAnnexB(GetParam(), &buf, nullptr));
EXPECT_EQ(0u, buf.size());
}
@@ -333,7 +332,7 @@ TEST_F(AVCConversionTest, ConvertConfigToAnnexB) {
avc_config.pps_list[0].push_back(0x56);
avc_config.pps_list[0].push_back(0x78);
- std::vector<uint8> buf;
+ std::vector<uint8_t> buf;
std::vector<SubsampleEntry> subsamples;
EXPECT_TRUE(AVC::ConvertConfigToAnnexB(avc_config, &buf));
EXPECT_EQ(0, memcmp(kExpectedParamSets, &buf[0],
@@ -345,7 +344,7 @@ TEST_F(AVCConversionTest, ConvertConfigToAnnexB) {
TEST_F(AVCConversionTest, StringConversionFunctions) {
std::string str =
"AUD SPS SPSExt SPS PPS SEI SEI R14 I P FILL EOSeq EOStr";
- std::vector<uint8> buf;
+ std::vector<uint8_t> buf;
std::vector<SubsampleEntry> subsamples;
StringToAnnexB(str, &buf, &subsamples);
EXPECT_TRUE(AVC::IsValidAnnexB(buf, subsamples));
@@ -375,7 +374,7 @@ TEST_F(AVCConversionTest, ValidAnnexBConstructs) {
};
for (size_t i = 0; i < arraysize(test_cases); ++i) {
- std::vector<uint8> buf;
+ std::vector<uint8_t> buf;
std::vector<SubsampleEntry> subsamples;
StringToAnnexB(test_cases[i], &buf, NULL);
EXPECT_TRUE(AVC::IsValidAnnexB(buf, subsamples)) << "'" << test_cases[i]
@@ -400,7 +399,7 @@ TEST_F(AVCConversionTest, InvalidAnnexBConstructs) {
};
for (size_t i = 0; i < arraysize(test_cases); ++i) {
- std::vector<uint8> buf;
+ std::vector<uint8_t> buf;
std::vector<SubsampleEntry> subsamples;
StringToAnnexB(test_cases[i], &buf, NULL);
EXPECT_FALSE(AVC::IsValidAnnexB(buf, subsamples)) << "'" << test_cases[i]
@@ -441,7 +440,7 @@ TEST_F(AVCConversionTest, InsertParamSetsAnnexB) {
avc_config.pps_list[0].push_back(0x78);
for (size_t i = 0; i < arraysize(test_cases); ++i) {
- std::vector<uint8> buf;
+ std::vector<uint8_t> buf;
std::vector<SubsampleEntry> subsamples;
StringToAnnexB(test_cases[i].input, &buf, &subsamples);

Powered by Google App Engine
This is Rietveld 408576698