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

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

Issue 242013002: Refactor video capturing code in the render process (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge again :( Created 6 years, 8 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); 109 mock_source()->DeliverVideoFrame(frame, media::VideoCaptureFormat());
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());
422 } 423 }
423 EXPECT_EQ(1, sink.number_of_frames()); 424 EXPECT_EQ(1, sink.number_of_frames());
424 // Expect the delivered frame to be passed unchanged since its smaller than 425 // Expect the delivered frame to be passed unchanged since its smaller than
425 // max requested. 426 // max requested.
426 EXPECT_EQ(320, sink.frame_size().width()); 427 EXPECT_EQ(320, sink.frame_size().width());
427 EXPECT_EQ(240, sink.frame_size().height()); 428 EXPECT_EQ(240, sink.frame_size().height());
428 429
429 { 430 {
430 scoped_refptr<media::VideoFrame> frame2 = 431 scoped_refptr<media::VideoFrame> frame2 =
431 media::VideoFrame::CreateBlackFrame(gfx::Size(640, 480)); 432 media::VideoFrame::CreateBlackFrame(gfx::Size(640, 480));
432 mock_source()->DeliverVideoFrame(frame2); 433 mock_source()->DeliverVideoFrame(frame2,
434 media::VideoCaptureFormat());
433 } 435 }
434 EXPECT_EQ(2, sink.number_of_frames()); 436 EXPECT_EQ(2, sink.number_of_frames());
435 // Expect the delivered frame to be passed unchanged since its smaller than 437 // Expect the delivered frame to be passed unchanged since its smaller than
436 // max requested. 438 // max requested.
437 EXPECT_EQ(640, sink.frame_size().width()); 439 EXPECT_EQ(640, sink.frame_size().width());
438 EXPECT_EQ(480, sink.frame_size().height()); 440 EXPECT_EQ(480, sink.frame_size().height());
439 441
440 { 442 {
441 scoped_refptr<media::VideoFrame> frame3 = 443 scoped_refptr<media::VideoFrame> frame3 =
442 media::VideoFrame::CreateBlackFrame(gfx::Size(1280, 720)); 444 media::VideoFrame::CreateBlackFrame(gfx::Size(1280, 720));
443 mock_source()->DeliverVideoFrame(frame3); 445 mock_source()->DeliverVideoFrame(frame3,
446 media::VideoCaptureFormat());
444 } 447 }
445 EXPECT_EQ(3, sink.number_of_frames()); 448 EXPECT_EQ(3, sink.number_of_frames());
446 // Expect a frame to be cropped since its larger than max requested. 449 // Expect a frame to be cropped since its larger than max requested.
447 EXPECT_EQ(800, sink.frame_size().width()); 450 EXPECT_EQ(800, sink.frame_size().width());
448 EXPECT_EQ(700, sink.frame_size().height()); 451 EXPECT_EQ(700, sink.frame_size().height());
449 452
450 MediaStreamVideoSink::RemoveFromVideoTrack(&sink, track); 453 MediaStreamVideoSink::RemoveFromVideoTrack(&sink, track);
451 } 454 }
452 455
453 TEST_F(MediaStreamVideoSourceTest, IsConstraintSupported) { 456 TEST_F(MediaStreamVideoSourceTest, IsConstraintSupported) {
(...skipping 12 matching lines...) Expand all
466 EXPECT_TRUE(MediaStreamVideoSource::IsConstraintSupported( 469 EXPECT_TRUE(MediaStreamVideoSource::IsConstraintSupported(
467 MediaStreamVideoSource::kMaxAspectRatio)); 470 MediaStreamVideoSource::kMaxAspectRatio));
468 EXPECT_TRUE(MediaStreamVideoSource::IsConstraintSupported( 471 EXPECT_TRUE(MediaStreamVideoSource::IsConstraintSupported(
469 MediaStreamVideoSource::kMinAspectRatio)); 472 MediaStreamVideoSource::kMinAspectRatio));
470 473
471 EXPECT_FALSE(MediaStreamVideoSource::IsConstraintSupported( 474 EXPECT_FALSE(MediaStreamVideoSource::IsConstraintSupported(
472 "something unsupported")); 475 "something unsupported"));
473 } 476 }
474 477
475 } // namespace content 478 } // 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