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

Side by Side Diff: content/browser/media/media_canplaytype_browsertest.cc

Issue 812643005: Re-add AC3/EAC3 audio demuxing support (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed unit tests Created 5 years, 9 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 unified diff | Download patch
« no previous file with comments | « build/common.gypi ('k') | media/base/audio_decoder_config.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <string> 5 #include <string>
6 6
7 #include "content/browser/media/media_browsertest.h" 7 #include "content/browser/media/media_browsertest.h"
8 #include "content/public/test/browser_test_utils.h" 8 #include "content/public/test/browser_test_utils.h"
9 #include "content/public/test/content_browser_test_utils.h" 9 #include "content/public/test/content_browser_test_utils.h"
10 #include "content/shell/browser/shell.h" 10 #include "content/shell/browser/shell.h"
(...skipping 154 matching lines...) Expand 10 before | Expand all | Expand 10 after
165 CanPlay("'" + mime + "; codecs=\", AVC3.64001F, MP4.40.02\"'")); 165 CanPlay("'" + mime + "; codecs=\", AVC3.64001F, MP4.40.02\"'"));
166 166
167 // Unknown codecs. 167 // Unknown codecs.
168 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"avc2\"'")); 168 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"avc2\"'"));
169 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"avc4\"'")); 169 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"avc4\"'"));
170 170
171 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"avc1x\"'")); 171 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"avc1x\"'"));
172 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"avc3x\"'")); 172 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"avc3x\"'"));
173 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"mp4ax\"'")); 173 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"mp4ax\"'"));
174 174
175 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"ac-2\"'"));
176 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"ec-2\"'"));
177 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"ac-4\"'"));
ddorwin 2015/03/02 19:35:38 nit: add ec-4 for completeness/consistency
servolk 2015/11/24 19:45:17 Done.
178 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"mp4a.a4\"'"));
179 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"mp4a.a7\"'"));
180 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"mp4a.a5.1\"'"));
ddorwin 2015/03/02 19:35:38 Add: mp4a.a5. (ends in period)
servolk 2015/11/24 19:45:17 Done.
181
175 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"unknown\"'")); 182 EXPECT_EQ(kNot, CanPlay("'" + mime + "; codecs=\"unknown\"'"));
176 } 183 }
177 184
178 void TestOGGUnacceptableCombinations(const std::string& mime) { 185 void TestOGGUnacceptableCombinations(const std::string& mime) {
179 // Codecs not belonging to OGG container. 186 // Codecs not belonging to OGG container.
180 EXPECT_EQ(kNot, CanPlay("'" + mime +"; codecs=\"vp8\"'")); 187 EXPECT_EQ(kNot, CanPlay("'" + mime +"; codecs=\"vp8\"'"));
181 EXPECT_EQ(kNot, CanPlay("'" + mime +"; codecs=\"vp8.0\"'")); 188 EXPECT_EQ(kNot, CanPlay("'" + mime +"; codecs=\"vp8.0\"'"));
182 EXPECT_EQ(kNot, CanPlay("'" + mime +"; codecs=\"vp8, opus\"'")); 189 EXPECT_EQ(kNot, CanPlay("'" + mime +"; codecs=\"vp8, opus\"'"));
183 EXPECT_EQ(kNot, CanPlay("'" + mime +"; codecs=\"vp8, vorbis\"'")); 190 EXPECT_EQ(kNot, CanPlay("'" + mime +"; codecs=\"vp8, vorbis\"'"));
184 191
(...skipping 849 matching lines...) Expand 10 before | Expand all | Expand 10 after
1034 EXPECT_EQ(maybeCanPlayHLS, 1041 EXPECT_EQ(maybeCanPlayHLS,
1035 CanPlay("'application/vnd.apple.mpegurl; " 1042 CanPlay("'application/vnd.apple.mpegurl; "
1036 "codecs=\"avc1.42E01E, mp4a.40\"'")); 1043 "codecs=\"avc1.42E01E, mp4a.40\"'"));
1037 EXPECT_EQ(maybeCanPlayHLS, 1044 EXPECT_EQ(maybeCanPlayHLS,
1038 CanPlay("'application/vnd.apple.mpegurl; " 1045 CanPlay("'application/vnd.apple.mpegurl; "
1039 "codecs=\"avc3.42E01E, mp4a.40\"'")); 1046 "codecs=\"avc3.42E01E, mp4a.40\"'"));
1040 1047
1041 TestMPEGUnacceptableCombinations("application/vnd.apple.mpegurl"); 1048 TestMPEGUnacceptableCombinations("application/vnd.apple.mpegurl");
1042 } 1049 }
1043 1050
1051 IN_PROC_BROWSER_TEST_F(MediaCanPlayTypeTest, CodecSupportTest_ac3_eac3) {
1052 const char* expected_result = kNot;
ddorwin 2015/03/02 19:35:38 You'll probably need to move this up to line 20.
servolk 2015/11/24 19:45:17 Done.
1053 #if defined(ENABLE_AC3_EAC3_AUDIO_DEMUXING)
1054 expected_result = kProbably;
1055 #endif
1056 // AC3
1057 EXPECT_EQ(expected_result, CanPlay("'audio/mp4; codecs=\"ac-3\"'"));
1058 EXPECT_EQ(expected_result, CanPlay("'audio/mp4; codecs=\"mp4a.a5\"'"));
1059 EXPECT_EQ(expected_result, CanPlay("'video/mp4; codecs=\"ac-3\"'"));
1060 EXPECT_EQ(expected_result, CanPlay("'video/mp4; codecs=\"mp4a.a5\"'"));
1061 // EAC3 aka DD+ aka Dolby Digital Plus
1062 EXPECT_EQ(expected_result, CanPlay("'audio/mp4; codecs=\"ec-3\"'"));
1063 EXPECT_EQ(expected_result, CanPlay("'audio/mp4; codecs=\"mp4a.a6\"'"));
1064 EXPECT_EQ(expected_result, CanPlay("'video/mp4; codecs=\"ec-3\"'"));
1065 EXPECT_EQ(expected_result, CanPlay("'video/mp4; codecs=\"mp4a.a6\"'"));
1066 }
1067
1044 } // namespace content 1068 } // namespace content
OLDNEW
« no previous file with comments | « build/common.gypi ('k') | media/base/audio_decoder_config.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698