MediaRecorder: Tune complexity settings for VP9 encoding

content/renderer/media/video_track_recorder_unittest.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 <stddef.h>
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
@@ skipping 20 matching lines @@
 using ::testing::SaveArg;
 using ::testing::TestWithParam;
 using ::testing::ValuesIn;
 
 namespace content {
 
 ACTION_P(RunClosure, closure) {
   closure.Run();
 }
 
-struct TrackRecorderTestParams {
-  const bool use_vp9;
-  const size_t first_encoded_frame_size;
-  const size_t second_encoded_frame_size;
-  const size_t third_encoded_frame_size;
-};
+const bool kTrackRecorderTestUseVp9OrNot[] = {false, true};
 
-const TrackRecorderTestParams kTrackRecorderTestParams[] = {{false, 52, 32, 57},
-                                                            {true, 33, 18, 33}};
-
-class VideoTrackRecorderTest : public TestWithParam<TrackRecorderTestParams> {
+class VideoTrackRecorderTest : public TestWithParam<bool> {
  public:
   VideoTrackRecorderTest()
       : mock_source_(new MockMediaStreamVideoSource(false)) {
     const blink::WebString webkit_track_id(base::UTF8ToUTF16("dummy"));
     blink_source_.initialize(webkit_track_id,
                              blink::WebMediaStreamSource::TypeVideo,
                              webkit_track_id);
     blink_source_.setExtraData(mock_source_);
     blink_track_.initialize(blink_source_);
 
     blink::WebMediaConstraints constraints;
     constraints.initialize();
     track_ = new MediaStreamVideoTrack(mock_source_, constraints,
                                        MediaStreamSource::ConstraintsCallback(),
                                        true /* enabled */);
     blink_track_.setExtraData(track_);
 
     video_track_recorder_.reset(new VideoTrackRecorder(
-        GetParam().use_vp9 /* use_vp9 */, blink_track_,
+        GetParam() /* use_vp9 */, blink_track_,
         base::Bind(&VideoTrackRecorderTest::OnEncodedVideo,
                    base::Unretained(this)),
         0 /* bits_per_second */));
     // Paranoia checks.
     EXPECT_EQ(blink_track_.source().extraData(), blink_source_.extraData());
     EXPECT_TRUE(message_loop_.IsCurrent());
   }
 
   ~VideoTrackRecorderTest() {
     blink_track_.reset();
@@ skipping 80 matching lines @@
 
   base::StringPiece third_frame_encoded_data;
   EXPECT_CALL(*this, DoOnEncodedVideo(video_frame2, _, _, true))
       .Times(1)
       .WillOnce(DoAll(SaveArg<1>(&third_frame_encoded_data),
                 RunClosure(quit_closure)));
   Encode(video_frame2, base::TimeTicks::Now());
 
   run_loop.Run();
 
-  EXPECT_EQ(GetParam().first_encoded_frame_size,
-            first_frame_encoded_data.size());
-  EXPECT_EQ(GetParam().second_encoded_frame_size,
-            second_frame_encoded_data.size());
-  EXPECT_EQ(GetParam().third_encoded_frame_size,
-            third_frame_encoded_data.size());
+  const size_t kEncodedSizeThreshold = 18;
+  EXPECT_GE(first_frame_encoded_data.size(), kEncodedSizeThreshold);
+  EXPECT_GE(second_frame_encoded_data.size(), kEncodedSizeThreshold);
+  EXPECT_GE(third_frame_encoded_data.size(), kEncodedSizeThreshold);
 
   Mock::VerifyAndClearExpectations(this);
 }
 
 INSTANTIATE_TEST_CASE_P(,
                         VideoTrackRecorderTest,
-                        ValuesIn(kTrackRecorderTestParams));
+                        ValuesIn(kTrackRecorderTestUseVp9OrNot));
 
 }  // namespace content