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

Unified Diff: content/renderer/media_recorder/video_track_recorder.h

Issue 2801803002: Android: enable H264&VP8 HW accelerator for MediaRecorder (Closed)
Patch Set: remove change in ffmpeg_demuxer.cc Created 3 years, 8 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 side-by-side diff with in-line comments
Download patch
Index: content/renderer/media_recorder/video_track_recorder.h
diff --git a/content/renderer/media_recorder/video_track_recorder.h b/content/renderer/media_recorder/video_track_recorder.h
index 53a871f32512a7e4a4a23462fa26e568ac1c6fcc..046b1adcaeeae22f02f30044ea1e68799215f5bb 100644
--- a/content/renderer/media_recorder/video_track_recorder.h
+++ b/content/renderer/media_recorder/video_track_recorder.h
@@ -18,6 +18,12 @@
#include "third_party/WebKit/public/platform/WebMediaStreamTrack.h"
#include "third_party/skia/include/core/SkBitmap.h"
+#if BUILDFLAG(RTC_USE_H264) || defined(OS_ANDROID)
+// H264 encoding is supported on Android using the platform encode
+// accelerator, and elsewhere using OpenH264.
+#define IS_H264_SUPPORTED
+#endif
+
namespace base {
class Thread;
} // namespace base
@@ -32,8 +38,13 @@ class VideoFrame;
} // namespace media
namespace video_track_recorder {
+#if defined(OS_ANDROID)
+const int kVEAEncoderMinResolutionWidth = 176;
+const int kVEAEncoderMinResolutionHeight = 144;
+#else
const int kVEAEncoderMinResolutionWidth = 640;
const int kVEAEncoderMinResolutionHeight = 480;
+#endif
} // namespace video_track_recorder
namespace content {
@@ -51,7 +62,7 @@ class CONTENT_EXPORT VideoTrackRecorder
enum class CodecId {
VP8,
VP9,
-#if BUILDFLAG(RTC_USE_H264)
+#if defined(IS_H264_SUPPORTED)
H264,
#endif
LAST
« no previous file with comments | « content/renderer/media_recorder/media_recorder_handler.cc ('k') | content/renderer/media_recorder/video_track_recorder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698