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

Unified Diff: media/filters/h264_parser_unittest.cc

Issue 119153002: Move H264Parser and H264BitReader to media/filters. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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
« no previous file with comments | « media/filters/h264_parser.cc ('k') | media/media.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/filters/h264_parser_unittest.cc
diff --git a/content/common/gpu/media/h264_parser_unittest.cc b/media/filters/h264_parser_unittest.cc
similarity index 57%
rename from content/common/gpu/media/h264_parser_unittest.cc
rename to media/filters/h264_parser_unittest.cc
index d52bb5122094f519f409cd453af2245393092326..aec3bd5209327f755e6e5f9688bf77b3765940e7 100644
--- a/content/common/gpu/media/h264_parser_unittest.cc
+++ b/media/filters/h264_parser_unittest.cc
@@ -2,28 +2,25 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "testing/gtest/include/gtest/gtest.h"
-
#include "base/command_line.h"
#include "base/files/memory_mapped_file.h"
#include "base/logging.h"
+#include "base/path_service.h"
#include "base/strings/string_number_conversions.h"
-#include "content/common/gpu/media/h264_parser.h"
-
-using content::H264Parser;
-using content::H264NALU;
+#include "media/base/test_data_util.h"
+#include "media/filters/h264_parser.h"
+#include "testing/gtest/include/gtest/gtest.h"
-const base::FilePath::CharType* test_stream_filename =
- FILE_PATH_LITERAL("content/common/gpu/testdata/test-25fps.h264");
-// Number of NALUs in the stream to be parsed.
-int num_nalus = 759;
+namespace media {
TEST(H264ParserTest, StreamFileParsing) {
- base::FilePath fp(test_stream_filename);
+ base::FilePath file_path = GetTestDataFilePath("test-25fps.h264");
+ // Number of NALUs in the test stream to be parsed.
+ int num_nalus = 759;
+
base::MemoryMappedFile stream;
- CHECK(stream.Initialize(fp)) << "Couldn't open stream file: "
- << test_stream_filename;
- DVLOG(1) << "Parsing file: " << test_stream_filename;
+ ASSERT_TRUE(stream.Initialize(file_path))
+ << "Couldn't open stream file: " << file_path.MaybeAsASCII();
H264Parser parser;
parser.SetStream(stream.data(), stream.length());
@@ -31,8 +28,8 @@ TEST(H264ParserTest, StreamFileParsing) {
// Parse until the end of stream/unsupported stream/error in stream is found.
int num_parsed_nalus = 0;
while (true) {
- content::H264SliceHeader shdr;
- content::H264SEIMessage sei_msg;
+ media::H264SliceHeader shdr;
+ media::H264SEIMessage sei_msg;
H264NALU nalu;
H264Parser::Result res = parser.AdvanceToNextNALU(&nalu);
if (res == H264Parser::kEOStream) {
@@ -72,22 +69,4 @@ TEST(H264ParserTest, StreamFileParsing) {
}
}
-int main(int argc, char **argv) {
- ::testing::InitGoogleTest(&argc, argv);
- CommandLine::Init(argc, argv);
-
- const CommandLine::SwitchMap& switches =
- CommandLine::ForCurrentProcess()->GetSwitches();
- for (CommandLine::SwitchMap::const_iterator it = switches.begin();
- it != switches.end(); ++it) {
- if (it->first == "test_stream") {
- test_stream_filename = it->second.c_str();
- } else if (it->first == "num_nalus") {
- CHECK(base::StringToInt(it->second, &num_nalus));
- } else {
- LOG(FATAL) << "Unexpected switch: " << it->first << ":" << it->second;
- }
- }
-
- return RUN_ALL_TESTS();
-}
+} // namespace media
« no previous file with comments | « media/filters/h264_parser.cc ('k') | media/media.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698