VideoCapture: more migration IPC-->mojo, part 5

content/renderer/media/video_capture_impl_unittest.cc

 // Copyright (c) 2012 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 <stddef.h>
 
 #include "base/macros.h"
 #include "base/memory/shared_memory.h"
 #include "base/message_loop/message_loop.h"
 #include "content/child/child_process.h"
-#include "content/common/media/video_capture_messages.h"
 #include "content/common/video_capture.mojom.h"
 #include "content/renderer/media/video_capture_impl.h"
-#include "media/base/bind_to_current_loop.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using ::testing::_;
 using ::testing::Invoke;
 using ::testing::InvokeWithoutArgs;
 using ::testing::SaveArg;
 using ::testing::WithArgs;
 
 namespace content {
 
 const int kSessionId = 1;
 
 void RunEmptyFormatsCallback(const VideoCaptureDeviceFormatsCB& callback) {
   media::VideoCaptureFormats formats;
   callback.Run(formats);
 }
 
-// Mock implementation of the Mojo service. TODO(mcasas): Replace completely
-// MockVideoCaptureMessageFilter, https://crbug.com/651897
+// Mock implementation of the Mojo Host service.
 class MockMojoVideoCaptureHost : public mojom::VideoCaptureHost {
  public:
   MockMojoVideoCaptureHost() : released_buffer_count_(0) {
     ON_CALL(*this, GetDeviceSupportedFormats(_, _, _))
         .WillByDefault(WithArgs<2>(Invoke(RunEmptyFormatsCallback)));
     ON_CALL(*this, GetDeviceFormatsInUse(_, _, _))
         .WillByDefault(WithArgs<2>(Invoke(RunEmptyFormatsCallback)));
     ON_CALL(*this, ReleaseBuffer(_, _, _, _))
         .WillByDefault(InvokeWithoutArgs(
             this, &MockMojoVideoCaptureHost::increase_released_buffer_count));
@@ skipping 24 matching lines @@
 
   int released_buffer_count() const { return released_buffer_count_; }
   void increase_released_buffer_count() { released_buffer_count_++; }
 
  private:
   int released_buffer_count_;
 
   DISALLOW_COPY_AND_ASSIGN(MockMojoVideoCaptureHost);
 };
 
-class MockVideoCaptureMessageFilter : public VideoCaptureMessageFilter {
- public:
-  MockVideoCaptureMessageFilter() : VideoCaptureMessageFilter() {}
-
-  // Filter implementation.
-  MOCK_METHOD1(Send, bool(IPC::Message* message));
-
- protected:
-  virtual ~MockVideoCaptureMessageFilter() {}
-
- private:
-  DISALLOW_COPY_AND_ASSIGN(MockVideoCaptureMessageFilter);
-};
-
+// This class encapsulates a VideoCaptureImpl under test and the necessary
+// accessory classes, namely:
+// - a VideoCaptureMessageFilter;
+// - a MockVideoCaptureHost, mimicking the RendererHost;
+// - a few callbacks that are bound when calling operations of VideoCaptureImpl
+//  and on which we set expectations.
 class VideoCaptureImplTest : public ::testing::Test {
  public:
   VideoCaptureImplTest()
-      : message_filter_(new MockVideoCaptureMessageFilter),
+      : message_filter_(new VideoCaptureMessageFilter),
         video_capture_impl_(
             new VideoCaptureImpl(kSessionId,
                                  message_filter_.get(),
                                  base::ThreadTaskRunnerHandle::Get())) {
     params_small_.requested_format = media::VideoCaptureFormat(
         gfx::Size(176, 144), 30, media::PIXEL_FORMAT_I420);
     params_large_.requested_format = media::VideoCaptureFormat(
         gfx::Size(320, 240), 30, media::PIXEL_FORMAT_I420);
 
     video_capture_impl_->SetVideoCaptureHostForTesting(
         &mock_video_capture_host_);
     video_capture_impl_->device_id_ = 2;
   }
 
  protected:
   // These four mocks are used to create callbacks for the different oeprations.
   MOCK_METHOD2(OnFrameReady,
                void(const scoped_refptr<media::VideoFrame>&, base::TimeTicks));
   MOCK_METHOD1(OnStateUpdate, void(VideoCaptureState));
   MOCK_METHOD1(OnDeviceFormatsInUse, void(const media::VideoCaptureFormats&));
   MOCK_METHOD1(OnDeviceSupportedFormats,
                void(const media::VideoCaptureFormats&));
 
   void StartCapture(int client_id, const media::VideoCaptureParams& params) {
-    video_capture_impl_->StartCapture(
-        client_id, params, base::Bind(&VideoCaptureImplTest::OnStateUpdate,
-                                      base::Unretained(this)),
-        base::Bind(&VideoCaptureImplTest::OnFrameReady,
-                   base::Unretained(this)));
+    const auto state_update_callback = base::Bind(
+        &VideoCaptureImplTest::OnStateUpdate, base::Unretained(this));
+    const auto frame_ready_callback =
+        base::Bind(&VideoCaptureImplTest::OnFrameReady, base::Unretained(this));
+
+    video_capture_impl_->StartCapture(client_id, params, state_update_callback,
+                                      frame_ready_callback);
   }
 
   void StopCapture(int client_id) {
     video_capture_impl_->StopCapture(client_id);
   }
 
   void NewBuffer(int buffer_id, const base::SharedMemory& shm) {
     video_capture_impl_->OnBufferCreated(
         base::SharedMemory::DuplicateHandle(shm.handle()),
         shm.mapped_size(), buffer_id);
   }
 
   void BufferReceived(int buffer_id, const gfx::Size& size) {
-    base::TimeTicks now = base::TimeTicks::Now();
-    base::TimeDelta timestamp = now - base::TimeTicks();
+    mojom::VideoFrameInfoPtr info = mojom::VideoFrameInfo::New();
 
+    const base::TimeTicks now = base::TimeTicks::Now();
     media::VideoFrameMetadata frame_metadata;
     frame_metadata.SetTimeTicks(media::VideoFrameMetadata::REFERENCE_TIME, now);
-    base::DictionaryValue metadata;
-    frame_metadata.MergeInternalValuesInto(&metadata);
+    frame_metadata.MergeInternalValuesInto(&info->metadata);
 
-    video_capture_impl_->OnBufferReceived(
-        buffer_id, timestamp, metadata, media::PIXEL_FORMAT_I420,
-        media::VideoFrame::STORAGE_SHMEM, size, gfx::Rect(size));
+    info->timestamp = now - base::TimeTicks();
+    info->pixel_format = media::mojom::VideoFormat::I420;
+    info->storage_type = media::PIXEL_STORAGE_CPU;
+    info->coded_size = size;
+    info->visible_rect = gfx::Rect(size);
+
+    video_capture_impl_->OnBufferReady(buffer_id, std::move(info));
   }
 
   void BufferDestroyed(int buffer_id) {
     video_capture_impl_->OnBufferDestroyed(buffer_id);
   }
 
   void GetDeviceSupportedFormats() {
     const base::Callback<void(const media::VideoCaptureFormats&)>
         callback = base::Bind(
             &VideoCaptureImplTest::OnDeviceSupportedFormats,
             base::Unretained(this));
     video_capture_impl_->GetDeviceSupportedFormats(callback);
   }
 
   void GetDeviceFormatsInUse() {
     const base::Callback<void(const media::VideoCaptureFormats&)>
         callback = base::Bind(
             &VideoCaptureImplTest::OnDeviceFormatsInUse,
             base::Unretained(this));
     video_capture_impl_->GetDeviceFormatsInUse(callback);
   }
 
   void OnStateChanged(mojom::VideoCaptureState state) {
     video_capture_impl_->OnStateChanged(state);
   }
 
   const base::MessageLoop message_loop_;
   const ChildProcess child_process_;
-  const scoped_refptr<MockVideoCaptureMessageFilter> message_filter_;
+  const scoped_refptr<VideoCaptureMessageFilter> message_filter_;
   const std::unique_ptr<VideoCaptureImpl> video_capture_impl_;
   MockMojoVideoCaptureHost mock_video_capture_host_;
   media::VideoCaptureParams params_small_;
   media::VideoCaptureParams params_large_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(VideoCaptureImplTest);
 };
 
 TEST_F(VideoCaptureImplTest, Simple) {
@@ skipping 68 matching lines @@
 // provided callback.
 TEST_F(VideoCaptureImplTest, GetDeviceFormatsInUse) {
   EXPECT_CALL(*this, OnDeviceFormatsInUse(_));
   EXPECT_CALL(mock_video_capture_host_,
               GetDeviceFormatsInUse(_, kSessionId, _));
 
   GetDeviceFormatsInUse();
 }
 
 TEST_F(VideoCaptureImplTest, BufferReceived) {
+  const int kBufferId = 11;
+
   base::SharedMemory shm;
   const size_t frame_size = media::VideoFrame::AllocationSize(
       media::PIXEL_FORMAT_I420, params_small_.requested_format.frame_size);
   ASSERT_TRUE(shm.CreateAndMapAnonymous(frame_size));
 
   EXPECT_CALL(*this, OnStateUpdate(VIDEO_CAPTURE_STATE_STARTED));
   EXPECT_CALL(*this, OnStateUpdate(VIDEO_CAPTURE_STATE_STOPPED));
   EXPECT_CALL(*this, OnFrameReady(_, _));
   EXPECT_CALL(mock_video_capture_host_, DoStart(_, kSessionId, params_small_));
   EXPECT_CALL(mock_video_capture_host_, Stop(_));
-  EXPECT_CALL(mock_video_capture_host_, ReleaseBuffer(_, _, _, _)).Times(0);
+  EXPECT_CALL(mock_video_capture_host_, ReleaseBuffer(_, kBufferId, _, _))
+      .Times(0);
 
   StartCapture(0, params_small_);
-  NewBuffer(0, shm);
-  BufferReceived(0, params_small_.requested_format.frame_size);
+  NewBuffer(kBufferId, shm);
+  BufferReceived(kBufferId, params_small_.requested_format.frame_size);
   StopCapture(0);
-  BufferDestroyed(0);
+  BufferDestroyed(kBufferId);
 
   EXPECT_EQ(mock_video_capture_host_.released_buffer_count(), 0);
 }
 
 TEST_F(VideoCaptureImplTest, BufferReceivedAfterStop) {
+  const int kBufferId = 12;
+
   base::SharedMemory shm;
   const size_t frame_size = media::VideoFrame::AllocationSize(
       media::PIXEL_FORMAT_I420, params_large_.requested_format.frame_size);
   ASSERT_TRUE(shm.CreateAndMapAnonymous(frame_size));
 
   EXPECT_CALL(*this, OnStateUpdate(VIDEO_CAPTURE_STATE_STARTED));
   EXPECT_CALL(*this, OnStateUpdate(VIDEO_CAPTURE_STATE_STOPPED));
   EXPECT_CALL(*this, OnFrameReady(_, _)).Times(0);
   EXPECT_CALL(mock_video_capture_host_, DoStart(_, kSessionId, params_large_));
   EXPECT_CALL(mock_video_capture_host_, Stop(_));
-  EXPECT_CALL(mock_video_capture_host_, ReleaseBuffer(_, _, _, _));
+  EXPECT_CALL(mock_video_capture_host_, ReleaseBuffer(_, kBufferId, _, _));
 
   StartCapture(0, params_large_);
-  NewBuffer(0, shm);
+  NewBuffer(kBufferId, shm);
   StopCapture(0);
   // A buffer received after StopCapture() triggers an instant ReleaseBuffer().
-  BufferReceived(0, params_large_.requested_format.frame_size);
-  BufferDestroyed(0);
+  BufferReceived(kBufferId, params_large_.requested_format.frame_size);
+  BufferDestroyed(kBufferId);
 
   EXPECT_EQ(mock_video_capture_host_.released_buffer_count(), 1);
 }
 
 TEST_F(VideoCaptureImplTest, AlreadyStarted) {
   media::VideoCaptureParams params = {};
   EXPECT_CALL(*this, OnStateUpdate(VIDEO_CAPTURE_STATE_STARTED)).Times(2);
   EXPECT_CALL(*this, OnStateUpdate(VIDEO_CAPTURE_STATE_STOPPED)).Times(2);
   EXPECT_CALL(mock_video_capture_host_, DoStart(_, kSessionId, _))
       .WillOnce(SaveArg<2>(&params));
@@ skipping 24 matching lines @@
   EXPECT_CALL(mock_video_capture_host_, DoStart(_, kSessionId, params_small_));
 
   StartCapture(0, params_small_);
 
   OnStateChanged(mojom::VideoCaptureState::FAILED);
 
   StopCapture(0);
 }
 
 }  // namespace content