Android: enable H264&VP8 HW accelerator for MediaRecorder

content/renderer/media_recorder/media_recorder_handler.cc

Index: content/renderer/media_recorder/media_recorder_handler.cc
diff --git a/content/renderer/media_recorder/media_recorder_handler.cc b/content/renderer/media_recorder/media_recorder_handler.cc
index 9cc45602ffe7e4ce4d5168815c1848651ac9a9eb..df05b17ced55f4c0a1ccaac6d84b1a1a5687e1fe 100644
--- a/content/renderer/media_recorder/media_recorder_handler.cc
+++ b/content/renderer/media_recorder/media_recorder_handler.cc
@@ -40,7 +40,7 @@ media::VideoCodec CodecIdToMediaVideoCodec(VideoTrackRecorder::CodecId id) {
       return media::kCodecVP8;
     case VideoTrackRecorder::CodecId::VP9:
       return media::kCodecVP9;
-#if BUILDFLAG(RTC_USE_H264)
+#if BUILDFLAG(RTC_USE_H264) || defined(OS_ANDROID)

[emircan, 2017/04/07 17:33:04]

Can you add these checks to video_track_recorder_unittest,
media_recorder_handler_unittest as well so that tests run for android too?

[braveyao, 2017/04/08 00:56:26]

Done.

[braveyao, 2017/04/14 22:28:28]

Can't add the checks to video_track_recorder_unittest, which won't enumerate HW
codecs. See the comments added there.
Anyway, it's tested in media_recorder_browsertest.

     case VideoTrackRecorder::CodecId::H264:
       return media::kCodecH264;
 #endif
@@ -133,7 +133,7 @@ bool MediaRecorderHandler::initialize(
     codec_id_ = VideoTrackRecorder::CodecId::VP8;
   else if (codecs_str.find("vp9") != std::string::npos)
     codec_id_ = VideoTrackRecorder::CodecId::VP9;
-#if BUILDFLAG(RTC_USE_H264)
+#if BUILDFLAG(RTC_USE_H264) || defined(OS_ANDROID)
   else if (codecs_str.find("h264") != std::string::npos)
     codec_id_ = VideoTrackRecorder::CodecId::H264;
   else if (codecs_str.find("avc1") != std::string::npos)