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

Side by Side Diff: media/base/video_decoder_config.cc

Issue 1469353010: Configure MediaCodec with CDM in ADVA (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase, used std::move, added VDA::Config::AsHumanReadableString Created 5 years 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 // 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 "media/base/video_decoder_config.h" 5 #include "media/base/video_decoder_config.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "media/base/video_frame.h" 10 #include "media/base/video_frame.h"
(...skipping 122 matching lines...) Expand 10 before | Expand all | Expand 10 after
133 return "theora"; 133 return "theora";
134 case kCodecVP8: 134 case kCodecVP8:
135 return "vp8"; 135 return "vp8";
136 case kCodecVP9: 136 case kCodecVP9:
137 return "vp9"; 137 return "vp9";
138 } 138 }
139 NOTREACHED(); 139 NOTREACHED();
140 return ""; 140 return "";
141 } 141 }
142 142
143 // static
144 std::string VideoDecoderConfig::GetHumanReadableProfile(
145 VideoCodecProfile profile) {
146 // Keep this switch() in sync with VideoCodecProfile definition in
147 // media/base/video_codecs.h
xhwang 2015/12/07 22:14:15 If you move this to media/base/video_codecs.h then
Tima Vaisburd 2015/12/08 02:12:24 Done.
148 switch (profile) {
149 case VIDEO_CODEC_PROFILE_UNKNOWN:
150 return "unknown";
151 case H264PROFILE_BASELINE:
152 return "h264 baseline";
153 case H264PROFILE_MAIN:
154 return "h264 main";
155 case H264PROFILE_EXTENDED:
156 return "h264 extended";
157 case H264PROFILE_HIGH:
158 return "h264 high";
159 case H264PROFILE_HIGH10PROFILE:
160 return "h264 high 10";
161 case H264PROFILE_HIGH422PROFILE:
162 return "h264 high 4:2:2";
163 case H264PROFILE_HIGH444PREDICTIVEPROFILE:
164 return "h264 high 4:4:4 predictive";
165 case H264PROFILE_SCALABLEBASELINE:
166 return "h264 scalable baseline";
167 case H264PROFILE_SCALABLEHIGH:
168 return "h264 scalable high";
169 case H264PROFILE_STEREOHIGH:
170 return "h264 stereo high";
171 case H264PROFILE_MULTIVIEWHIGH:
172 return "h264 multiview high";
173 case VP8PROFILE_ANY:
174 return "vp8 profile";
xhwang 2015/12/07 22:14:15 "profile" seems redundant.
Tima Vaisburd 2015/12/08 02:12:24 Done.
175 case VP9PROFILE_ANY:
176 return "vp9 profile";
xhwang 2015/12/07 22:14:15 ditto
Tima Vaisburd 2015/12/08 02:12:24 Done.
177 }
178 NOTREACHED();
179 return "";
180 }
181
143 } // namespace media 182 } // namespace media
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698