MediaRecorder: parameterise certain browsertests by video codec and/or encode accelerator

content/browser/media/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/public/common/content_switches.h"
 #include "content/public/test/browser_test_utils.h"
 #include "content/public/test/content_browser_test_utils.h"
 #include "content/test/webrtc_content_browsertest_base.h"
 #include "media/base/media_switches.h"
 
 namespace {
 
 static const char kMediaRecorderHtmlFile[] = "/media/mediarecorder_test.html";
 
+static struct EncodingParameters {
+  bool disable_accelerator;
+  std::string video_codec;
+} const kEncodingParameters[] = {
+    {true, "VP8"},  {true, "VP9"},  {true, "H264"},
+    {false, "VP8"}, {false, "VP9"}, {false, "H264"},
+};
+
 }  // namespace
 
 namespace content {
 
 // This class tests the recording of a media stream.
-class WebRtcMediaRecorderTest : public WebRtcContentBrowserTest {
+class WebRtcMediaRecorderTest
+    : public WebRtcContentBrowserTest,
+      public testing::WithParamInterface<struct EncodingParameters> {
  public:
   WebRtcMediaRecorderTest() {}
   ~WebRtcMediaRecorderTest() override {}
 
   void SetUpCommandLine(base::CommandLine* command_line) override {
     WebRtcContentBrowserTest::SetUpCommandLine(command_line);
 
     AppendUseFakeUIForMediaStreamFlag();
 
     base::CommandLine::ForCurrentProcess()->AppendSwitch(
         switches::kUseFakeDeviceForMediaStream);
 
     base::CommandLine::ForCurrentProcess()->AppendSwitchASCII(
         switches::kEnableBlinkFeatures, "GetUserMedia");
   }
 
+  void MaybeForceDisableEncodeAccelerator(bool disable) {
+    if (!disable)

[phoglund_chromium, 2016/06/02 08:46:23]

You may as well do the GetParam in here, no? Seems like it will read better and
be less clutter.

[mcasas, 2016/06/02 14:57:29]

GetParam() only works if the test case has been
instantiated with an _P() macro, otherwise it hits
https://code.google.com/p/chromium/codesearch#chromium/src/testing/gtest/incl...

+      return;
+    // This flag is also used for encoding, https://crbug.com/616640.
+    base::CommandLine::ForCurrentProcess()->AppendSwitch(
+        switches::kDisableAcceleratedVideoDecode);
+  }
+
  private:
   DISALLOW_COPY_AND_ASSIGN(WebRtcMediaRecorderTest);
 };
 
 IN_PROC_BROWSER_TEST_F(WebRtcMediaRecorderTest, MediaRecorderStart) {
   MakeTypicalCall("testStartAndRecorderState();", kMediaRecorderHtmlFile);
 }
 
 IN_PROC_BROWSER_TEST_F(WebRtcMediaRecorderTest, MediaRecorderStartAndStop) {
   MakeTypicalCall("testStartStopAndRecorderState();", kMediaRecorderHtmlFile);
 }
 
-IN_PROC_BROWSER_TEST_F(WebRtcMediaRecorderTest,
+IN_PROC_BROWSER_TEST_P(WebRtcMediaRecorderTest,
                        MediaRecorderStartAndDataAvailable) {
-  MakeTypicalCall("testStartAndDataAvailable();", kMediaRecorderHtmlFile);
+  MaybeForceDisableEncodeAccelerator(GetParam().disable_accelerator);
+  MakeTypicalCall(base::StringPrintf("testStartAndDataAvailable(\"%s\");",
+                                     GetParam().video_codec.c_str()),
+                  kMediaRecorderHtmlFile);
 }
 
-IN_PROC_BROWSER_TEST_F(WebRtcMediaRecorderTest,
+IN_PROC_BROWSER_TEST_P(WebRtcMediaRecorderTest,
                        MediaRecorderStartWithTimeSlice) {
-  MakeTypicalCall("testStartWithTimeSlice();", kMediaRecorderHtmlFile);
+  MaybeForceDisableEncodeAccelerator(GetParam().disable_accelerator);
+  MakeTypicalCall(base::StringPrintf("testStartWithTimeSlice(\"%s\");",
+                                     GetParam().video_codec.c_str()),
+                  kMediaRecorderHtmlFile);
 }
 
 IN_PROC_BROWSER_TEST_F(WebRtcMediaRecorderTest, MediaRecorderResume) {
   MakeTypicalCall("testResumeAndRecorderState();", kMediaRecorderHtmlFile);
 }
 
 IN_PROC_BROWSER_TEST_F(WebRtcMediaRecorderTest,
                        MediaRecorderNoResumeWhenRecorderInactive) {
   MakeTypicalCall("testIllegalResumeThrowsDOMError();", kMediaRecorderHtmlFile);
 }
 
-IN_PROC_BROWSER_TEST_F(WebRtcMediaRecorderTest,
+IN_PROC_BROWSER_TEST_P(WebRtcMediaRecorderTest,
                        MediaRecorderResumeAndDataAvailable) {
-  MakeTypicalCall("testResumeAndDataAvailable();", kMediaRecorderHtmlFile);
+  MaybeForceDisableEncodeAccelerator(GetParam().disable_accelerator);
+  MakeTypicalCall(base::StringPrintf("testResumeAndDataAvailable(\"%s\");",
+                                     GetParam().video_codec.c_str()),
+                  kMediaRecorderHtmlFile);
 }
 
 IN_PROC_BROWSER_TEST_F(WebRtcMediaRecorderTest, MediaRecorderPause) {
   MakeTypicalCall("testPauseAndRecorderState();", kMediaRecorderHtmlFile);
 }
 
 IN_PROC_BROWSER_TEST_F(WebRtcMediaRecorderTest, MediaRecorderPauseStop) {
   MakeTypicalCall("testPauseStopAndRecorderState();", kMediaRecorderHtmlFile);
 }
 
@@ skipping 54 matching lines @@
   MakeTypicalCall("testAddingTrackToMediaStreamFiresErrorEvent();",
                   kMediaRecorderHtmlFile);
 }
 
 IN_PROC_BROWSER_TEST_F(WebRtcMediaRecorderTest,
                        RemovingTrackFromMediaStreamFiresErrorEvent) {
   MakeTypicalCall("testRemovingTrackFromMediaStreamFiresErrorEvent();",
                   kMediaRecorderHtmlFile);
 }
 
+INSTANTIATE_TEST_CASE_P(,

[phoglund_chromium, 2016/06/02 08:46:23]

Hmm, this looks strange, but if it works I'm ok with it.

[mcasas, 2016/06/02 14:57:29]

Acknowledged.

+                        WebRtcMediaRecorderTest,
+                        testing::ValuesIn(kEncodingParameters));
+
 }  // namespace content