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

Side by Side Diff: webrtc/api/mediatypes.cc

Issue 2675173003: Adding "adapter" ORTC objects on top of ChannelManager/BaseChannel/etc. (Closed)
Patch Set: 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
OLDNEW
1 /* 1 /*
2 * Copyright 2016 The WebRTC project authors. All Rights Reserved. 2 * Copyright 2016 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
11 #include "webrtc/api/mediatypes.h" 11 #include "webrtc/api/mediatypes.h"
12 #include "webrtc/base/checks.h" 12 #include "webrtc/base/checks.h"
13 13
14 namespace cricket { 14 namespace cricket {
15 15
16 std::string MediaTypeToString(MediaType type) { 16 std::string MediaTypeToString(MediaType type) {
17 std::string type_str;
18 switch (type) { 17 switch (type) {
19 case MEDIA_TYPE_AUDIO: 18 case MEDIA_TYPE_AUDIO:
20 type_str = "audio"; 19 return "audio";
21 break;
22 case MEDIA_TYPE_VIDEO: 20 case MEDIA_TYPE_VIDEO:
23 type_str = "video"; 21 return "video";
24 break;
25 case MEDIA_TYPE_DATA: 22 case MEDIA_TYPE_DATA:
26 type_str = "data"; 23 return "data";
pthatcher1 2017/02/08 01:33:48 It doesn't matter too much, but it seems like maki
Taylor Brandstetter 2017/02/10 00:19:45 Done.
27 break;
28 default:
29 RTC_NOTREACHED();
30 break;
31 } 24 }
32 return type_str; 25 }
26
27 MediaType MediaTypeFromString(const std::string& type_str) {
28 if (type_str == "audio") {
29 return MEDIA_TYPE_AUDIO;
30 } else if (type_str == "video") {
31 return MEDIA_TYPE_VIDEO;
32 } else if (type_str == "data") {
33 return MEDIA_TYPE_DATA;
34 } else {
35 FATAL();
36 }
33 } 37 }
34 38
35 } // namespace cricket 39 } // namespace cricket
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698