Make MediaStreamVideoTrack::GetSettings return the track's frame rate.

content/renderer/media/media_stream_video_track_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 <stdint.h>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/test/scoped_feature_list.h"
 #include "base/threading/thread_checker_impl.h"
 #include "content/child/child_process.h"
 #include "content/public/common/content_features.h"
 #include "content/renderer/media/media_stream_video_track.h"
 #include "content/renderer/media/mock_media_stream_video_sink.h"
 #include "content/renderer/media/mock_media_stream_video_source.h"
+#include "content/renderer/media/video_track_adapter.h"
 #include "media/base/video_frame.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "third_party/WebKit/public/web/WebHeap.h"
 
 namespace content {
 
 const uint8_t kBlackValue = 0x00;
 const uint8_t kColorValue = 0xAB;
 const int kMockSourceWidth = 640;
 const int kMockSourceHeight = 480;
@@ skipping 58 matching lines @@
     const bool enabled = true;
     blink::WebMediaStreamTrack track = MediaStreamVideoTrack::CreateVideoTrack(
         mock_source_, MediaStreamSource::ConstraintsCallback(), enabled);
     if (!source_started_) {
       mock_source_->StartMockedSource();
       source_started_ = true;
     }
     return track;
   }
 
+  // Create a track that's associated with |mock_source_| and has the given
+  // |adapter_settings|.
+  blink::WebMediaStreamTrack CreateTrackWithSettings(
+      const VideoTrackAdapterSettings& adapter_settings) {
+    const bool enabled = true;
+    blink::WebMediaStreamTrack track = MediaStreamVideoTrack::CreateVideoTrack(
+        mock_source_, adapter_settings, base::Optional<bool>(), false, 0.0,
+        MediaStreamSource::ConstraintsCallback(), enabled);
+    if (!source_started_) {
+      mock_source_->StartMockedSource();
+      source_started_ = true;
+    }
+    return track;
+  }
+
   void UpdateVideoSourceToRespondToRequestRefreshFrame() {
     blink_source_.Reset();
     mock_source_ = IsOldVideoConstraints()
                        ? new MockMediaStreamVideoSource(false, true)
                        : new MockMediaStreamVideoSource(
                              media::VideoCaptureFormat(
                                  gfx::Size(kMockSourceWidth, kMockSourceHeight),
                                  30.0, media::PIXEL_FORMAT_I420),
                              true);
     blink_source_.Initialize(blink::WebString::FromASCII("dummy_source_id"),
@@ skipping 181 matching lines @@
   blink::WebMediaStreamTrack::Settings settings;
   native_track->GetSettings(settings);
   // These values come straight from the mock video track implementation.
   EXPECT_EQ(640, settings.width);
   EXPECT_EQ(480, settings.height);
   EXPECT_EQ(30.0, settings.frame_rate);
   EXPECT_EQ(blink::WebMediaStreamTrack::FacingMode::kNone,
             settings.facing_mode);
 }
 
+TEST_F(MediaStreamVideoTrackTest, GetSettingsWithAdjustment) {
+  InitializeSource();
+  const int kAdjustedWidth = 600;
+  const int kAdjustedHeight = 400;
+  const double kAdjustedFrameRate = 20.0;
+  VideoTrackAdapterSettings adapter_settings(kAdjustedWidth, kAdjustedHeight,
+                                             0.0, 10000.0, kAdjustedFrameRate);
+  blink::WebMediaStreamTrack track = CreateTrackWithSettings(adapter_settings);
+  MediaStreamVideoTrack* const native_track =
+      MediaStreamVideoTrack::GetVideoTrack(track);
+  blink::WebMediaStreamTrack::Settings settings;
+  native_track->GetSettings(settings);
+  EXPECT_EQ(kAdjustedWidth, settings.width);
+  EXPECT_EQ(kAdjustedHeight, settings.height);
+  EXPECT_EQ(kAdjustedFrameRate, settings.frame_rate);
+  EXPECT_EQ(blink::WebMediaStreamTrack::FacingMode::kNone,
+            settings.facing_mode);
+}
+
+TEST_F(MediaStreamVideoTrackTest, GetSettingsStopped) {
+  InitializeSource();
+  blink::WebMediaStreamTrack track = CreateTrack();
+  MediaStreamVideoTrack* const native_track =
+      MediaStreamVideoTrack::GetVideoTrack(track);
+  native_track->Stop();
+  blink::WebMediaStreamTrack::Settings settings;
+  native_track->GetSettings(settings);
+  EXPECT_EQ(-1, settings.width);
+  EXPECT_EQ(-1, settings.height);
+  EXPECT_EQ(-1, settings.frame_rate);
+  EXPECT_EQ(blink::WebMediaStreamTrack::FacingMode::kNone,
+            settings.facing_mode);
+  EXPECT_TRUE(settings.device_id.IsNull());
+}
+
 // TODO(guidou): Remove this test. http://crbug.com/706408
 class MediaStreamVideoTrackOldConstraintsTest : public ::testing::Test {
  public:
   MediaStreamVideoTrackOldConstraintsTest()
       : child_process_(new ChildProcess()),
         mock_source_(nullptr),
         source_started_(false) {
     scoped_feature_list_.InitAndEnableFeature(
         features::kMediaStreamOldVideoConstraints);
   }
@@ skipping 219 matching lines @@
   native_track->GetSettings(settings);
   // These values come straight from the mock video track implementation.
   EXPECT_EQ(640, settings.width);
   EXPECT_EQ(480, settings.height);
   EXPECT_EQ(30.0, settings.frame_rate);
   EXPECT_EQ(blink::WebMediaStreamTrack::FacingMode::kNone,
             settings.facing_mode);
 }
 
 }  // namespace content