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

Side by Side Diff: content/browser/media_browsertest.cc

Issue 10452004: Add GSM_MS and PCM_ALAW codecs to CrOS FFMPEG. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 4 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "base/basictypes.h" 5 #include "base/basictypes.h"
6 #include "base/string16.h" 6 #include "base/string16.h"
7 #include "base/stringprintf.h" 7 #include "base/stringprintf.h"
8 #include "base/string_util.h" 8 #include "base/string_util.h"
9 #include "base/utf_string_conversions.h" 9 #include "base/utf_string_conversions.h"
10 #include "chrome/browser/ui/browser.h" 10 #include "chrome/browser/ui/browser.h"
(...skipping 77 matching lines...) Expand 10 before | Expand all | Expand 10 after
88 } 88 }
89 89
90 IN_PROC_BROWSER_TEST_F(MediaTest, VideoBear3gpAacH264) { 90 IN_PROC_BROWSER_TEST_F(MediaTest, VideoBear3gpAacH264) {
91 PlayVideo("bear_h264_aac.3gp"); 91 PlayVideo("bear_h264_aac.3gp");
92 } 92 }
93 93
94 IN_PROC_BROWSER_TEST_F(MediaTest, VideoBear3gpAmrnbMpeg4) { 94 IN_PROC_BROWSER_TEST_F(MediaTest, VideoBear3gpAmrnbMpeg4) {
95 PlayVideo("bear_mpeg4_amrnb.3gp"); 95 PlayVideo("bear_mpeg4_amrnb.3gp");
96 } 96 }
97 97
98 // TODO(ihf): Enable these audio codecs for CrOS. 98 IN_PROC_BROWSER_TEST_F(MediaTest, VideoBearWavGsmms) {
99 // IN_PROC_BROWSER_TEST_F(MediaTest, VideoBearWavAlaw) { 99 PlayVideo("bear_gsm_ms.wav");
DaleCurtis 2012/07/25 18:22:14 PlayAudio?
ilja 2012/07/25 20:09:37 Guilty of copy and pasting.
100 // PlayVideo("bear_alaw.wav"); 100 }
101 // }
102 // IN_PROC_BROWSER_TEST_F(MediaTest, VideoBearWavGsmms) {
103 // PlayVideo("bear_gsmms.wav");
104 // }
105 101
106 IN_PROC_BROWSER_TEST_F(MediaTest, VideoBearWavMulaw) { 102 IN_PROC_BROWSER_TEST_F(MediaTest, VideoBearWavMulaw) {
107 PlayAudio("bear_mulaw.wav"); 103 PlayAudio("bear_mulaw.wav");
108 } 104 }
109 105
106 IN_PROC_BROWSER_TEST_F(MediaTest, VideoBearMovPcmS16be) {
107 PlayAudio("bear_pcm_s16be.mov");
DaleCurtis 2012/07/25 18:22:14 PlayVideo? Why .mov instead of .wav files? Also I'
ilja 2012/07/25 20:09:37 FFmpeg refuses to encode into big endian wav files
108 }
109
110 IN_PROC_BROWSER_TEST_F(MediaTest, VideoBearMovPcmS24be) {
111 PlayAudio("bear_pcm_s24be.mov");
112 }
113
110 IN_PROC_BROWSER_TEST_F(MediaTest, VideoBearFlac) { 114 IN_PROC_BROWSER_TEST_F(MediaTest, VideoBearFlac) {
111 PlayAudio("bear.flac"); 115 PlayAudio("bear.flac");
112 } 116 }
113 #endif 117 #endif
114 #endif 118 #endif
115 119
116 IN_PROC_BROWSER_TEST_F(MediaTest, VideoBearWavPcm) { 120 IN_PROC_BROWSER_TEST_F(MediaTest, VideoBearWavPcm) {
117 PlayAudio("bear_pcm.wav"); 121 PlayAudio("bear_pcm.wav");
118 } 122 }
119 123
(...skipping 30 matching lines...) Expand all
150 } 154 }
151 155
152 // TODO(dalecurtis): Disabled because loop is flaky. http://crbug.com/134021 156 // TODO(dalecurtis): Disabled because loop is flaky. http://crbug.com/134021
153 // IN_PROC_BROWSER_TEST_F(MediaLayoutTest, VideoLoopTest) { 157 // IN_PROC_BROWSER_TEST_F(MediaLayoutTest, VideoLoopTest) {
154 // RunLayoutTest("video-loop.html"); 158 // RunLayoutTest("video-loop.html");
155 // } 159 // }
156 160
157 IN_PROC_BROWSER_TEST_F(MediaLayoutTest, VideoNoAutoplayTest) { 161 IN_PROC_BROWSER_TEST_F(MediaLayoutTest, VideoNoAutoplayTest) {
158 RunLayoutTest("video-no-autoplay.html"); 162 RunLayoutTest("video-no-autoplay.html");
159 } 163 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698