Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(609)

Side by Side Diff: content/renderer/media/media_stream_video_source_unittest.cc

Issue 263323003: Revert of Refactor video capturing code in the render process (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <string> 5 #include <string>
6 #include <vector> 6 #include <vector>
7 7
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "base/strings/string_number_conversions.h" 9 #include "base/strings/string_number_conversions.h"
10 #include "base/strings/utf_string_conversions.h" 10 #include "base/strings/utf_string_conversions.h"
(...skipping 88 matching lines...) Expand 10 before | Expand all | Expand 10 after
99 CreateTrackAndStartSource(constraints, capture_width, capture_height , 99 CreateTrackAndStartSource(constraints, capture_width, capture_height ,
100 30); 100 30);
101 101
102 MockMediaStreamVideoSink sink; 102 MockMediaStreamVideoSink sink;
103 MediaStreamVideoSink::AddToVideoTrack(&sink, track); 103 MediaStreamVideoSink::AddToVideoTrack(&sink, track);
104 EXPECT_EQ(0, sink.number_of_frames()); 104 EXPECT_EQ(0, sink.number_of_frames());
105 105
106 scoped_refptr<media::VideoFrame> frame = 106 scoped_refptr<media::VideoFrame> frame =
107 media::VideoFrame::CreateBlackFrame(gfx::Size(capture_width, 107 media::VideoFrame::CreateBlackFrame(gfx::Size(capture_width,
108 capture_height)); 108 capture_height));
109 mock_source()->DeliverVideoFrame(frame, media::VideoCaptureFormat()); 109 mock_source()->DeliverVideoFrame(frame);
110 EXPECT_EQ(1, sink.number_of_frames()); 110 EXPECT_EQ(1, sink.number_of_frames());
111 111
112 // Expect the delivered frame to be cropped. 112 // Expect the delivered frame to be cropped.
113 EXPECT_EQ(expected_height, sink.frame_size().height()); 113 EXPECT_EQ(expected_height, sink.frame_size().height());
114 EXPECT_EQ(expected_width, sink.frame_size().width()); 114 EXPECT_EQ(expected_width, sink.frame_size().width());
115 MediaStreamVideoSink::RemoveFromVideoTrack(&sink, track); 115 MediaStreamVideoSink::RemoveFromVideoTrack(&sink, track);
116 } 116 }
117 117
118 void ReleaseTrackAndSourceOnAddTrackCallback( 118 void ReleaseTrackAndSourceOnAddTrackCallback(
119 const blink::WebMediaStreamTrack& track_to_release) { 119 const blink::WebMediaStreamTrack& track_to_release) {
(...skipping 291 matching lines...) Expand 10 before | Expand all | Expand 10 after
411 CreateTrackAndStartSource(factory.CreateWebMediaConstraints(), 411 CreateTrackAndStartSource(factory.CreateWebMediaConstraints(),
412 640, 480, 30); 412 640, 480, 30);
413 413
414 MockMediaStreamVideoSink sink; 414 MockMediaStreamVideoSink sink;
415 MediaStreamVideoSink::AddToVideoTrack(&sink, track); 415 MediaStreamVideoSink::AddToVideoTrack(&sink, track);
416 EXPECT_EQ(0, sink.number_of_frames()); 416 EXPECT_EQ(0, sink.number_of_frames());
417 417
418 { 418 {
419 scoped_refptr<media::VideoFrame> frame1 = 419 scoped_refptr<media::VideoFrame> frame1 =
420 media::VideoFrame::CreateBlackFrame(gfx::Size(320, 240)); 420 media::VideoFrame::CreateBlackFrame(gfx::Size(320, 240));
421 mock_source()->DeliverVideoFrame(frame1, 421 mock_source()->DeliverVideoFrame(frame1);
422 media::VideoCaptureFormat());
423 } 422 }
424 EXPECT_EQ(1, sink.number_of_frames()); 423 EXPECT_EQ(1, sink.number_of_frames());
425 // Expect the delivered frame to be passed unchanged since its smaller than 424 // Expect the delivered frame to be passed unchanged since its smaller than
426 // max requested. 425 // max requested.
427 EXPECT_EQ(320, sink.frame_size().width()); 426 EXPECT_EQ(320, sink.frame_size().width());
428 EXPECT_EQ(240, sink.frame_size().height()); 427 EXPECT_EQ(240, sink.frame_size().height());
429 428
430 { 429 {
431 scoped_refptr<media::VideoFrame> frame2 = 430 scoped_refptr<media::VideoFrame> frame2 =
432 media::VideoFrame::CreateBlackFrame(gfx::Size(640, 480)); 431 media::VideoFrame::CreateBlackFrame(gfx::Size(640, 480));
433 mock_source()->DeliverVideoFrame(frame2, 432 mock_source()->DeliverVideoFrame(frame2);
434 media::VideoCaptureFormat());
435 } 433 }
436 EXPECT_EQ(2, sink.number_of_frames()); 434 EXPECT_EQ(2, sink.number_of_frames());
437 // Expect the delivered frame to be passed unchanged since its smaller than 435 // Expect the delivered frame to be passed unchanged since its smaller than
438 // max requested. 436 // max requested.
439 EXPECT_EQ(640, sink.frame_size().width()); 437 EXPECT_EQ(640, sink.frame_size().width());
440 EXPECT_EQ(480, sink.frame_size().height()); 438 EXPECT_EQ(480, sink.frame_size().height());
441 439
442 { 440 {
443 scoped_refptr<media::VideoFrame> frame3 = 441 scoped_refptr<media::VideoFrame> frame3 =
444 media::VideoFrame::CreateBlackFrame(gfx::Size(1280, 720)); 442 media::VideoFrame::CreateBlackFrame(gfx::Size(1280, 720));
445 mock_source()->DeliverVideoFrame(frame3, 443 mock_source()->DeliverVideoFrame(frame3);
446 media::VideoCaptureFormat());
447 } 444 }
448 EXPECT_EQ(3, sink.number_of_frames()); 445 EXPECT_EQ(3, sink.number_of_frames());
449 // Expect a frame to be cropped since its larger than max requested. 446 // Expect a frame to be cropped since its larger than max requested.
450 EXPECT_EQ(800, sink.frame_size().width()); 447 EXPECT_EQ(800, sink.frame_size().width());
451 EXPECT_EQ(700, sink.frame_size().height()); 448 EXPECT_EQ(700, sink.frame_size().height());
452 449
453 MediaStreamVideoSink::RemoveFromVideoTrack(&sink, track); 450 MediaStreamVideoSink::RemoveFromVideoTrack(&sink, track);
454 } 451 }
455 452
456 TEST_F(MediaStreamVideoSourceTest, IsConstraintSupported) { 453 TEST_F(MediaStreamVideoSourceTest, IsConstraintSupported) {
(...skipping 12 matching lines...) Expand all
469 EXPECT_TRUE(MediaStreamVideoSource::IsConstraintSupported( 466 EXPECT_TRUE(MediaStreamVideoSource::IsConstraintSupported(
470 MediaStreamVideoSource::kMaxAspectRatio)); 467 MediaStreamVideoSource::kMaxAspectRatio));
471 EXPECT_TRUE(MediaStreamVideoSource::IsConstraintSupported( 468 EXPECT_TRUE(MediaStreamVideoSource::IsConstraintSupported(
472 MediaStreamVideoSource::kMinAspectRatio)); 469 MediaStreamVideoSource::kMinAspectRatio));
473 470
474 EXPECT_FALSE(MediaStreamVideoSource::IsConstraintSupported( 471 EXPECT_FALSE(MediaStreamVideoSource::IsConstraintSupported(
475 "something unsupported")); 472 "something unsupported"));
476 } 473 }
477 474
478 } // namespace content 475 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/media/media_stream_video_source.cc ('k') | content/renderer/media/video_capture_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698