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

Side by Side Diff: content/renderer/media/recorder/media_recorder_handler.cc

Issue 2700893003: Various MimeUtil cleanups. (Closed)
Patch Set: Fix default codecs return Created 3 years, 10 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 | « no previous file | media/base/mime_util.h » ('j') | media/base/mime_util_internal.cc » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "content/renderer/media/recorder/media_recorder_handler.h" 5 #include "content/renderer/media/recorder/media_recorder_handler.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/location.h" 10 #include "base/location.h"
(...skipping 80 matching lines...) Expand 10 before | Expand all | Expand 10 after
91 // vp8, vp9, h264 and avc1 or opus; |type| = "audio", supports only opus. 91 // vp8, vp9, h264 and avc1 or opus; |type| = "audio", supports only opus.
92 // http://www.webmproject.org/docs/container Sec:"HTML5 Video Type Parameters" 92 // http://www.webmproject.org/docs/container Sec:"HTML5 Video Type Parameters"
93 static const char* const kVideoCodecs[] = {"vp8", "vp9", "h264", "avc1", 93 static const char* const kVideoCodecs[] = {"vp8", "vp9", "h264", "avc1",
94 "opus"}; 94 "opus"};
95 static const char* const kAudioCodecs[] = { "opus" }; 95 static const char* const kAudioCodecs[] = { "opus" };
96 const char* const* codecs = video ? &kVideoCodecs[0] : &kAudioCodecs[0]; 96 const char* const* codecs = video ? &kVideoCodecs[0] : &kAudioCodecs[0];
97 const int codecs_count = 97 const int codecs_count =
98 video ? arraysize(kVideoCodecs) : arraysize(kAudioCodecs); 98 video ? arraysize(kVideoCodecs) : arraysize(kAudioCodecs);
99 99
100 std::vector<std::string> codecs_list; 100 std::vector<std::string> codecs_list;
101 media::ParseCodecString(web_codecs.utf8(), &codecs_list, true /* strip */); 101 media::SplitCodecsToVector(web_codecs.utf8(), &codecs_list, true /* strip */);
102 for (const auto& codec : codecs_list) { 102 for (const auto& codec : codecs_list) {
103 auto* const* found = std::find_if( 103 auto* const* found = std::find_if(
104 &codecs[0], &codecs[codecs_count], [&codec](const char* name) { 104 &codecs[0], &codecs[codecs_count], [&codec](const char* name) {
105 return base::EqualsCaseInsensitiveASCII(codec, name); 105 return base::EqualsCaseInsensitiveASCII(codec, name);
106 }); 106 });
107 if (found == &codecs[codecs_count]) 107 if (found == &codecs[codecs_count])
108 return false; 108 return false;
109 } 109 }
110 return true; 110 return true;
111 } 111 }
(...skipping 215 matching lines...) Expand 10 before | Expand all | Expand 10 after
327 recorder->OnData(audio_bus, timestamp); 327 recorder->OnData(audio_bus, timestamp);
328 } 328 }
329 329
330 void MediaRecorderHandler::SetAudioFormatForTesting( 330 void MediaRecorderHandler::SetAudioFormatForTesting(
331 const media::AudioParameters& params) { 331 const media::AudioParameters& params) {
332 for (const auto& recorder : audio_recorders_) 332 for (const auto& recorder : audio_recorders_)
333 recorder->OnSetFormat(params); 333 recorder->OnSetFormat(params);
334 } 334 }
335 335
336 } // namespace content 336 } // namespace content
OLDNEW
« no previous file with comments | « no previous file | media/base/mime_util.h » ('j') | media/base/mime_util_internal.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698