Revert of [Mojo Video Capture] Hook up video capture service behind a feature flag

content/browser/renderer_host/media/video_capture_browsertest.cc

 // Copyright 2017 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/run_loop.h"
 #include "build/build_config.h"
 #include "content/browser/browser_main_loop.h"
 #include "content/browser/renderer_host/media/media_stream_manager.h"
 #include "content/browser/renderer_host/media/video_capture_controller.h"
 #include "content/browser/renderer_host/media/video_capture_manager.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/test/content_browser_test.h"
 #include "media/base/bind_to_current_loop.h"
 #include "media/base/media_switches.h"
 #include "media/capture/video_capture_types.h"
-#include "services/video_capture/public/cpp/constants.h"
 #include "testing/gmock/include/gmock/gmock.h"
 
 using testing::_;
 using testing::AtLeast;
-using testing::Bool;
-using testing::Combine;
 using testing::Invoke;
 using testing::InvokeWithoutArgs;
 using testing::Values;
 
 namespace content {
 
-static const char kFakeDeviceFactoryConfigString[] = "device-count=3";
-static const float kFrameRateToRequest = 15.0f;
-
 class MockVideoCaptureControllerEventHandler
     : public VideoCaptureControllerEventHandler {
  public:
   MOCK_METHOD4(DoOnBufferCreated,
                void(VideoCaptureControllerID id,
                     mojo::ScopedSharedBufferHandle* handle,
                     int length,
                     int buffer_id));
   MOCK_METHOD2(OnBufferDestroyed,
                void(VideoCaptureControllerID, int buffer_id));
@@ skipping 15 matching lines @@
   }
 };
 
 class MockMediaStreamProviderListener : public MediaStreamProviderListener {
  public:
   MOCK_METHOD2(Opened, void(MediaStreamType, int));
   MOCK_METHOD2(Closed, void(MediaStreamType, int));
   MOCK_METHOD2(Aborted, void(MediaStreamType, int));
 };
 
-using DeviceIndex = size_t;
-using Resolution = gfx::Size;
-using ExerciseAcceleratedJpegDecoding = bool;
-using UseMojoService = bool;
-
-// For converting the std::tuple<> used as test parameters back to something
-// human-readable.
 struct TestParams {
-  TestParams() : device_index_to_use(0u) {}
-  TestParams(const std::tuple<DeviceIndex,
-                              Resolution,
-                              ExerciseAcceleratedJpegDecoding,
-                              UseMojoService>& params)
-      : device_index_to_use(std::get<0>(params)),
-        resolution_to_use(std::get<1>(params)),
-        exercise_accelerated_jpeg_decoding(std::get<2>(params)),
-        use_mojo_service(std::get<3>(params)) {}
-
-  media::VideoPixelFormat GetPixelFormatToUse() {
-    return (device_index_to_use == 1u) ? media::PIXEL_FORMAT_Y16
-                                       : media::PIXEL_FORMAT_I420;
-  }
-
+  std::string fake_device_factory_config_string;
   size_t device_index_to_use;
+  media::VideoPixelFormat pixel_format_to_use;
   gfx::Size resolution_to_use;
+  float frame_rate_to_use;
   bool exercise_accelerated_jpeg_decoding;
-  bool use_mojo_service;
 };
 
 struct FrameInfo {
   gfx::Size size;
   media::VideoPixelFormat pixel_format;
   media::VideoPixelStorage storage_type;
   base::TimeDelta timestamp;
 };
 
-// Integration test that exercises the VideoCaptureManager instance running in
-// the Browser process.
-class VideoCaptureBrowserTest : public ContentBrowserTest,
-                                public ::testing::WithParamInterface<
-                                    std::tuple<DeviceIndex,
-                                               Resolution,
-                                               ExerciseAcceleratedJpegDecoding,
-                                               UseMojoService>> {
+class VideoCaptureBrowserTest
+    : public ContentBrowserTest,
+      public ::testing::WithParamInterface<TestParams> {
  public:
-  VideoCaptureBrowserTest() { params_ = TestParams(GetParam()); }
-
   void SetUpAndStartCaptureDeviceOnIOThread(base::Closure continuation) {
     video_capture_manager_ = media_stream_manager_->video_capture_manager();
     ASSERT_TRUE(video_capture_manager_);
     video_capture_manager_->RegisterListener(&mock_stream_provider_listener_);
     video_capture_manager_->EnumerateDevices(
         base::Bind(&VideoCaptureBrowserTest::OnDeviceDescriptorsReceived,
                    base::Unretained(this), std::move(continuation)));
   }
 
   void TearDownCaptureDeviceOnIOThread(base::Closure continuation,
                                        bool post_to_end_of_message_queue) {
-    // DisconnectClient() must not be called synchronously from either the
+    // StopCaptureForClient must not be called synchronously from either the
     // |done_cb| passed to StartCaptureForClient() nor any callback made to a
     // VideoCaptureControllerEventHandler. To satisfy this, we have to post our
     // invocation to the end of the IO message queue.
     if (post_to_end_of_message_queue) {
       base::ThreadTaskRunnerHandle::Get()->PostTask(
           FROM_HERE,
           base::Bind(&VideoCaptureBrowserTest::TearDownCaptureDeviceOnIOThread,
                      base::Unretained(this), continuation, false));
       return;
     }
 
     video_capture_manager_->DisconnectClient(controller_.get(), stub_client_id_,
                                              &mock_controller_event_handler_,
                                              false);
 
     EXPECT_CALL(mock_stream_provider_listener_, Closed(_, _))
         .WillOnce(InvokeWithoutArgs([continuation]() { continuation.Run(); }));
 
     video_capture_manager_->Close(session_id_);
   }
 
  protected:
   void SetUpCommandLine(base::CommandLine* command_line) override {
-    command_line->AppendSwitchASCII(switches::kUseFakeDeviceForMediaStream,
-                                    kFakeDeviceFactoryConfigString);
+    command_line->AppendSwitchASCII(
+        switches::kUseFakeDeviceForMediaStream,
+        GetParam().fake_device_factory_config_string);
     command_line->AppendSwitch(switches::kUseFakeUIForMediaStream);
-    if (params_.exercise_accelerated_jpeg_decoding) {
+    if (GetParam().exercise_accelerated_jpeg_decoding) {
       base::CommandLine::ForCurrentProcess()->AppendSwitch(
           switches::kUseFakeJpegDecodeAccelerator);
     } else {
       base::CommandLine::ForCurrentProcess()->AppendSwitch(
           switches::kDisableAcceleratedMjpegDecode);
     }
-    if (params_.use_mojo_service) {
-      base::CommandLine::ForCurrentProcess()->AppendSwitchASCII(
-          switches::kEnableFeatures, video_capture::kMojoVideoCapture.name);
-    }
   }
 
   // This cannot be part of an override of SetUp(), because at the time when
   // SetUp() is invoked, the BrowserMainLoop does not exist yet.
   void SetUpRequiringBrowserMainLoopOnMainThread() {
     BrowserMainLoop* browser_main_loop = BrowserMainLoop::GetInstance();
     ASSERT_TRUE(browser_main_loop);
     media_stream_manager_ = browser_main_loop->media_stream_manager();
     ASSERT_TRUE(media_stream_manager_);
   }
 
   void OnDeviceDescriptorsReceived(
       base::Closure continuation,
       const media::VideoCaptureDeviceDescriptors& descriptors) {
-    ASSERT_TRUE(params_.device_index_to_use < descriptors.size());
-    const auto& descriptor = descriptors[params_.device_index_to_use];
+    ASSERT_TRUE(GetParam().device_index_to_use < descriptors.size());
+    const auto& descriptor = descriptors[GetParam().device_index_to_use];
     MediaStreamDevice media_stream_device(
         MEDIA_DEVICE_VIDEO_CAPTURE, descriptor.device_id,
         descriptor.display_name, descriptor.facing);
     session_id_ = video_capture_manager_->Open(media_stream_device);
     media::VideoCaptureParams capture_params;
     capture_params.requested_format = media::VideoCaptureFormat(
-        params_.resolution_to_use, kFrameRateToRequest,
-        params_.GetPixelFormatToUse());
+        GetParam().resolution_to_use, GetParam().frame_rate_to_use,
+        GetParam().pixel_format_to_use);
     video_capture_manager_->ConnectClient(
         session_id_, capture_params, stub_client_id_,
         &mock_controller_event_handler_,
         base::Bind(&VideoCaptureBrowserTest::OnConnectClientToControllerAnswer,
                    base::Unretained(this), std::move(continuation)));
   }
 
   void OnConnectClientToControllerAnswer(
       base::Closure continuation,
       const base::WeakPtr<VideoCaptureController>& controller) {
     ASSERT_TRUE(controller.get());
     controller_ = controller;
     if (!continuation)
       return;
     continuation.Run();
   }
 
  protected:
-  TestParams params_;
   MediaStreamManager* media_stream_manager_ = nullptr;
   VideoCaptureManager* video_capture_manager_ = nullptr;
   int session_id_ = 0;
   const VideoCaptureControllerID stub_client_id_ = 123;
   MockMediaStreamProviderListener mock_stream_provider_listener_;
   MockVideoCaptureControllerEventHandler mock_controller_event_handler_;
   base::WeakPtr<VideoCaptureController> controller_;
 };
 
 IN_PROC_BROWSER_TEST_P(VideoCaptureBrowserTest, StartAndImmediatelyStop) {
-#if defined(OS_ANDROID)
-  // Mojo video capture is currently not supported on Android.
-  // TODO(chfremer): Remove this as soon as https://crbug.com/720500 is
-  // resolved.
-  if (params_.use_mojo_service)
-    return;
-#endif
-  // Mojo video capture currently does not support accelerated jpeg decoding.
-  // TODO(chfremer): Remove this as soon as https://crbug.com/720604 is
-  // resolved.
-  if (params_.use_mojo_service && params_.exercise_accelerated_jpeg_decoding)
-    return;
-
   SetUpRequiringBrowserMainLoopOnMainThread();
   base::RunLoop run_loop;
   auto quit_run_loop_on_current_thread_cb =
       media::BindToCurrentLoop(run_loop.QuitClosure());
   auto after_start_continuation =
       base::Bind(&VideoCaptureBrowserTest::TearDownCaptureDeviceOnIOThread,
                  base::Unretained(this),
                  std::move(quit_run_loop_on_current_thread_cb), true);
   BrowserThread::PostTask(
       content::BrowserThread::IO, FROM_HERE,
       base::Bind(&VideoCaptureBrowserTest::SetUpAndStartCaptureDeviceOnIOThread,
                  base::Unretained(this), std::move(after_start_continuation)));
   run_loop.Run();
 }
 
 IN_PROC_BROWSER_TEST_P(VideoCaptureBrowserTest,
                        ReceiveFramesFromFakeCaptureDevice) {
+// TODO(chfremer): This test case is flaky on Android. Find out cause of
+// flakiness and then re-enable. See crbug.com/709039.
 #if defined(OS_ANDROID)
-  // TODO(chfremer): This test case is flaky on Android. Find out cause of
-  // flakiness and then re-enable. See https://crbug.com/709039.
-  if (params_.exercise_accelerated_jpeg_decoding)
-    return;
-  // Mojo video capture is currently not supported on Android
-  // TODO(chfremer): Remove this as soon as https://crbug.com/720500 is
-  // resolved.
-  if (params_.use_mojo_service)
+  if (GetParam().exercise_accelerated_jpeg_decoding)
     return;
 #endif
-  // Mojo video capture currently does not support accelerated jpeg decoding.
-  // TODO(chfremer): Remove this as soon as https://crbug.com/720604 is
-  // resolved.
-  if (params_.use_mojo_service && params_.exercise_accelerated_jpeg_decoding)
-    return;
-  // Only fake device with index 2 delivers MJPEG.
-  if (params_.exercise_accelerated_jpeg_decoding &&
-      params_.device_index_to_use != 2)
-    return;
 
   SetUpRequiringBrowserMainLoopOnMainThread();
 
   std::vector<FrameInfo> received_frame_infos;
   static const size_t kMinFramesToReceive = 5;
   static const size_t kMaxFramesToReceive = 300;
   base::RunLoop run_loop;
 
   auto quit_run_loop_on_current_thread_cb =
       media::BindToCurrentLoop(run_loop.QuitClosure());
   auto finish_test_cb =
       base::Bind(&VideoCaptureBrowserTest::TearDownCaptureDeviceOnIOThread,
                  base::Unretained(this),
                  std::move(quit_run_loop_on_current_thread_cb), true);
 
   bool must_wait_for_gpu_decode_to_start = false;
-  if (params_.exercise_accelerated_jpeg_decoding) {
+  if (GetParam().exercise_accelerated_jpeg_decoding) {
     // Since the GPU jpeg decoder is created asynchronously while decoding
     // in software is ongoing, we have to keep pushing frames until a message
     // arrives that tells us that the GPU decoder is being used. Otherwise,
     // it may happen that all test frames are decoded using the non-GPU
     // decoding path before the GPU decoder has started getting used.
     must_wait_for_gpu_decode_to_start = true;
     EXPECT_CALL(mock_controller_event_handler_, OnStartedUsingGpuDecode(_))
         .WillOnce(InvokeWithoutArgs([&must_wait_for_gpu_decode_to_start]() {
           must_wait_for_gpu_decode_to_start = false;
         }));
@@ skipping 29 matching lines @@
       base::Bind(&VideoCaptureBrowserTest::SetUpAndStartCaptureDeviceOnIOThread,
                  base::Unretained(this), std::move(do_nothing)));
   run_loop.Run();
 
   EXPECT_FALSE(must_wait_for_gpu_decode_to_start);
   EXPECT_GE(received_frame_infos.size(), kMinFramesToReceive);
   EXPECT_LT(received_frame_infos.size(), kMaxFramesToReceive);
   base::TimeDelta previous_timestamp;
   bool first_frame = true;
   for (const auto& frame_info : received_frame_infos) {
-    EXPECT_EQ(params_.GetPixelFormatToUse(), frame_info.pixel_format);
+    EXPECT_EQ(GetParam().pixel_format_to_use, frame_info.pixel_format);
     EXPECT_EQ(media::PIXEL_STORAGE_CPU, frame_info.storage_type);
-    EXPECT_EQ(params_.resolution_to_use, frame_info.size);
+    EXPECT_EQ(GetParam().resolution_to_use, frame_info.size);
     // Timestamps are expected to increase
     if (!first_frame)
       EXPECT_GT(frame_info.timestamp, previous_timestamp);
     first_frame = false;
     previous_timestamp = frame_info.timestamp;
   }
 }
 
-INSTANTIATE_TEST_CASE_P(,
-                        VideoCaptureBrowserTest,
-                        Combine(Values(0, 1, 2),             // DeviceIndex
-                                Values(gfx::Size(640, 480),  // Resolution
-                                       gfx::Size(1280, 720)),
-                                Bool(),    // ExerciseAcceleratedJpegDecoding
-                                Bool()));  // UseMojoService
+INSTANTIATE_TEST_CASE_P(
+    ,
+    VideoCaptureBrowserTest,
+    Values(TestParams{"fps=25,device-count=2", 0, media::PIXEL_FORMAT_I420,
+                      gfx::Size(1280, 720), 25.0f, false},
+           // The 2nd device outputs Y16
+           TestParams{"fps=25,device-count=2", 1, media::PIXEL_FORMAT_Y16,
+                      gfx::Size(1280, 720), 25.0f, false},
+           TestParams{"fps=15,device-count=2", 1, media::PIXEL_FORMAT_Y16,
+                      gfx::Size(640, 480), 15.0f, false},
+           // The 3rd device outputs MJPEG, which is converted to I420.
+           TestParams{"fps=15,device-count=3", 2, media::PIXEL_FORMAT_I420,
+                      gfx::Size(640, 480), 25.0f, false},
+           TestParams{"fps=6,device-count=3", 2, media::PIXEL_FORMAT_I420,
+                      gfx::Size(640, 480), 6.0f, true}));
 
 }  // namespace content