New resolution change policies for desktop and tab capture.

content/renderer/media/media_stream_video_capture_source_unittest.cc

 // Copyright 2014 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/bind.h"
 #include "base/callback_helpers.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "base/strings/utf_string_conversions.h"
 #include "content/child/child_process.h"
@@ skipping 48 matching lines @@
     source_->SetDeviceInfo(device_info);
 
     webkit_source_.initialize(base::UTF8ToUTF16("dummy_source_id"),
                               blink::WebMediaStreamSource::TypeVideo,
                               base::UTF8ToUTF16("dummy_source_name"),
                               false /* remote */ , true /* readonly */);
     webkit_source_.setExtraData(source_);
     webkit_source_id_ = webkit_source_.id();
   }
 
+  MockMediaConstraintFactory* constraint_factory() {
+    return &constraint_factory_;
+  }
+
   blink::WebMediaStreamTrack StartSource() {
-    MockMediaConstraintFactory factory;
     bool enabled = true;
     // CreateVideoTrack will trigger OnConstraintsApplied.
     return MediaStreamVideoTrack::CreateVideoTrack(
-        source_, factory.CreateWebMediaConstraints(),
+        source_, constraint_factory_.CreateWebMediaConstraints(),
         base::Bind(
             &MediaStreamVideoCapturerSourceTest::OnConstraintsApplied,
             base::Unretained(this)),
             enabled);
   }
 
   MockVideoCapturerDelegate& mock_delegate() {
     return *delegate_;
   }
 
   void OnSourceStopped(const blink::WebMediaStreamSource& source) {
     source_stopped_ =  true;
     EXPECT_EQ(source.id(), webkit_source_id_);
   }
 
  protected:
   void OnConstraintsApplied(MediaStreamSource* source,
                             MediaStreamRequestResult result,
                             const blink::WebString& result_name) {
   }
 
   base::MessageLoopForUI message_loop_;
   scoped_ptr<ChildProcess> child_process_;
   blink::WebMediaStreamSource webkit_source_;
   MediaStreamVideoCapturerSource* source_;  // owned by |webkit_source_|.
   MockVideoCapturerDelegate* delegate_; // owned by |source|.
   blink::WebString webkit_source_id_;
   bool source_stopped_;
+  MockMediaConstraintFactory constraint_factory_;
 };
 
-TEST_F(MediaStreamVideoCapturerSourceTest, TabCaptureAllowResolutionChange) {
+TEST_F(MediaStreamVideoCapturerSourceTest, TabCaptureFixedResolutionByDefault) {
   StreamDeviceInfo device_info;
   device_info.device.type = MEDIA_TAB_VIDEO_CAPTURE;
   InitWithDeviceInfo(device_info);
 
+  // No constraints are being provided to the implementation, so expect only
+  // default values.
+  media::VideoCaptureParams expected_params;
+  expected_params.requested_format.frame_size.SetSize(
+      MediaStreamVideoSource::kDefaultWidth,
+      MediaStreamVideoSource::kDefaultHeight);
+  expected_params.requested_format.frame_rate =
+      MediaStreamVideoSource::kDefaultFrameRate;
+  expected_params.requested_format.pixel_format = media::PIXEL_FORMAT_I420;
+  expected_params.resolution_change_policy =
+      media::RESOLUTION_POLICY_FIXED_RESOLUTION;
+
   EXPECT_CALL(mock_delegate(), StartCapture(
-      testing::Field(&media::VideoCaptureParams::resolution_change_policy,
-                     media::RESOLUTION_POLICY_DYNAMIC_WITHIN_LIMIT),
+      expected_params,
       testing::_,
       testing::_,
       testing::_)).Times(1);
   blink::WebMediaStreamTrack track = StartSource();
   // When the track goes out of scope, the source will be stopped.
   EXPECT_CALL(mock_delegate(), StopCapture());
 }
 
 TEST_F(MediaStreamVideoCapturerSourceTest,
-       DesktopCaptureAllowResolutionChange) {
+       DesktopCaptureAllowAnyResolutionChangeByDefault) {
   StreamDeviceInfo device_info;
   device_info.device.type = MEDIA_DESKTOP_VIDEO_CAPTURE;
   InitWithDeviceInfo(device_info);
 
+  // No constraints are being provided to the implementation, so expect only
+  // default values.
+  media::VideoCaptureParams expected_params;
+  expected_params.requested_format.frame_size.SetSize(
+      MediaStreamVideoSource::kDefaultWidth,
+      MediaStreamVideoSource::kDefaultHeight);
+  expected_params.requested_format.frame_rate =
+      MediaStreamVideoSource::kDefaultFrameRate;
+  expected_params.requested_format.pixel_format = media::PIXEL_FORMAT_I420;
+  expected_params.resolution_change_policy =
+      media::RESOLUTION_POLICY_ANY_WITHIN_LIMIT;
+
   EXPECT_CALL(mock_delegate(), StartCapture(
-      testing::Field(&media::VideoCaptureParams::resolution_change_policy,
-                     media::RESOLUTION_POLICY_DYNAMIC_WITHIN_LIMIT),
+      expected_params,
       testing::_,
       testing::_,
       testing::_)).Times(1);
+  blink::WebMediaStreamTrack track = StartSource();
+  // When the track goes out of scope, the source will be stopped.
+  EXPECT_CALL(mock_delegate(), StopCapture());
+}
+
+TEST_F(MediaStreamVideoCapturerSourceTest,
+       TabCaptureConstraintsImplyFixedAspectRatio) {
+  StreamDeviceInfo device_info;
+  device_info.device.type = MEDIA_TAB_VIDEO_CAPTURE;
+  InitWithDeviceInfo(device_info);
+
+  // Specify max and min size constraints that have the same ~16:9 aspect ratio.
+  constraint_factory()->AddMandatory(MediaStreamVideoSource::kMaxWidth, 1920);
+  constraint_factory()->AddMandatory(MediaStreamVideoSource::kMaxHeight, 1080);
+  constraint_factory()->AddMandatory(MediaStreamVideoSource::kMinWidth, 854);
+  constraint_factory()->AddMandatory(MediaStreamVideoSource::kMinHeight, 480);
+  constraint_factory()->AddMandatory(MediaStreamVideoSource::kMaxFrameRate,
+                                     60.0);
+
+  media::VideoCaptureParams expected_params;
+  expected_params.requested_format.frame_size.SetSize(1920, 1080);
+  expected_params.requested_format.frame_rate = 60.0;
+  expected_params.requested_format.pixel_format = media::PIXEL_FORMAT_I420;
+  expected_params.resolution_change_policy =
+      media::RESOLUTION_POLICY_FIXED_ASPECT_RATIO;
+
+  EXPECT_CALL(mock_delegate(), StartCapture(
+      testing::Field(&media::VideoCaptureParams::resolution_change_policy,
+                     media::RESOLUTION_POLICY_FIXED_ASPECT_RATIO),
+      testing::_,
+      testing::_,
+      testing::_)).Times(1);
+  blink::WebMediaStreamTrack track = StartSource();
+  // When the track goes out of scope, the source will be stopped.
+  EXPECT_CALL(mock_delegate(), StopCapture());
+}
+
+TEST_F(MediaStreamVideoCapturerSourceTest,
+       TabCaptureConstraintsImplyAllowingAnyResolutionChange) {
+  StreamDeviceInfo device_info;
+  device_info.device.type = MEDIA_TAB_VIDEO_CAPTURE;
+  InitWithDeviceInfo(device_info);
+
+  // Specify max and min size constraints with different aspect ratios.
+  constraint_factory()->AddMandatory(MediaStreamVideoSource::kMaxWidth, 1920);
+  constraint_factory()->AddMandatory(MediaStreamVideoSource::kMaxHeight, 1080);
+  constraint_factory()->AddMandatory(MediaStreamVideoSource::kMinWidth, 0);
+  constraint_factory()->AddMandatory(MediaStreamVideoSource::kMinHeight, 0);
+  constraint_factory()->AddMandatory(MediaStreamVideoSource::kMaxFrameRate,
+                                     60.0);
+
+  media::VideoCaptureParams expected_params;
+  expected_params.requested_format.frame_size.SetSize(1920, 1080);
+  expected_params.requested_format.frame_rate = 60.0;
+  expected_params.requested_format.pixel_format = media::PIXEL_FORMAT_I420;
+  expected_params.resolution_change_policy =
+      media::RESOLUTION_POLICY_ANY_WITHIN_LIMIT;
+
+  EXPECT_CALL(mock_delegate(), StartCapture(
+      expected_params,
+      testing::_,
+      testing::_,
+      testing::_)).Times(1);
   blink::WebMediaStreamTrack track = StartSource();
   // When the track goes out of scope, the source will be stopped.
   EXPECT_CALL(mock_delegate(), StopCapture());
 }
 
 TEST_F(MediaStreamVideoCapturerSourceTest, Ended) {
   StreamDeviceInfo device_info;
   device_info.device.type = MEDIA_DESKTOP_VIDEO_CAPTURE;
   scoped_ptr<VideoCapturerDelegate> delegate(
       new VideoCapturerDelegate(device_info));
@@ skipping 94 matching lines @@
   run_loop.Run();
   FakeMediaStreamVideoSink::RemoveFromVideoTrack(&fake_sink, track);
   EXPECT_EQ(reference_capture_time, capture_time);
   double metadata_value;
   EXPECT_TRUE(metadata.GetDouble(media::VideoFrameMetadata::FRAME_RATE,
                                  &metadata_value));
   EXPECT_EQ(30.0, metadata_value);
 }
 
 }  // namespace content