Android: enable H264&VP8 HW accelerator for MediaRecorder

content/browser/webrtc/webrtc_media_recorder_browsertest.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/command_line.h"
 #include "base/macros.h"
 #include "base/strings/stringprintf.h"
 #include "build/build_config.h"
 #include "content/browser/webrtc/webrtc_content_browsertest_base.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/test/browser_test_utils.h"
 #include "content/public/test/content_browser_test_utils.h"
 #include "media/base/media_switches.h"
 
 namespace {
 
 static const char kMediaRecorderHtmlFile[] = "/media/mediarecorder_test.html";
 
 static struct EncodingParameters {
   bool disable_accelerator;
   std::string mime_type;
 } const kEncodingParameters[] = {
     {true, "video/webm;codecs=VP8"},
     {true, "video/webm;codecs=VP9"},
+#if !defined(OS_ANDROID)

[mcasas, 2017/04/17 20:10:38]

Don't use negative conditions: they are hard to read.
Instead, do:

#if defined(RTC_USE_H264)
  {true, "video/x-matroska;codecs=AVC1"},
#endif

This way, if and when H264 sw is enabled for Android,
we'll pick it up automatically.

[braveyao, 2017/04/18 16:15:38]

Done.

+    // OpenH264 is not supported on Android.
     {true, "video/x-matroska;codecs=AVC1"},
+#endif
     {false, ""},  // Instructs the platform to choose any accelerated codec.
     {false, "video/webm;codecs=VP8"},
     {false, "video/webm;codecs=VP9"},
     {false, "video/x-matroska;codecs=AVC1"},
 };
 
 }  // namespace
 
 namespace content {
 
@@ skipping 148 matching lines @@
                        RemovingTrackFromMediaStreamFiresErrorEvent) {
   MakeTypicalCall("testRemovingTrackFromMediaStreamFiresErrorEvent();",
                   kMediaRecorderHtmlFile);
 }
 
 INSTANTIATE_TEST_CASE_P(,
                         WebRtcMediaRecorderTest,
                         testing::ValuesIn(kEncodingParameters));
 
 }  // namespace content