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

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

Issue 1915793002: Remove a |readonly| member of MediaStreamTrack (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove a |readonly| argument in unittest files Created 4 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
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 "content/renderer/media/media_stream_video_capturer_source.h" 5 #include "content/renderer/media/media_stream_video_capturer_source.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/callback_helpers.h" 10 #include "base/callback_helpers.h"
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
79 delegate_ = delegate.get(); 79 delegate_ = delegate.get();
80 source_ = new MediaStreamVideoCapturerSource( 80 source_ = new MediaStreamVideoCapturerSource(
81 base::Bind(&MediaStreamVideoCapturerSourceTest::OnSourceStopped, 81 base::Bind(&MediaStreamVideoCapturerSourceTest::OnSourceStopped,
82 base::Unretained(this)), 82 base::Unretained(this)),
83 std::move(delegate)); 83 std::move(delegate));
84 source_->SetDeviceInfo(device_info); 84 source_->SetDeviceInfo(device_info);
85 85
86 webkit_source_.initialize(base::UTF8ToUTF16("dummy_source_id"), 86 webkit_source_.initialize(base::UTF8ToUTF16("dummy_source_id"),
87 blink::WebMediaStreamSource::TypeVideo, 87 blink::WebMediaStreamSource::TypeVideo,
88 base::UTF8ToUTF16("dummy_source_name"), 88 base::UTF8ToUTF16("dummy_source_name"),
89 false /* remote */, 89 false /* remote */);
90 true /* readonly */);
91 webkit_source_.setExtraData(source_); 90 webkit_source_.setExtraData(source_);
92 webkit_source_id_ = webkit_source_.id(); 91 webkit_source_id_ = webkit_source_.id();
93 } 92 }
94 93
95 MockConstraintFactory* constraint_factory() { return &constraint_factory_; } 94 MockConstraintFactory* constraint_factory() { return &constraint_factory_; }
96 95
97 blink::WebMediaStreamTrack StartSource() { 96 blink::WebMediaStreamTrack StartSource() {
98 bool enabled = true; 97 bool enabled = true;
99 // CreateVideoTrack will trigger OnConstraintsApplied. 98 // CreateVideoTrack will trigger OnConstraintsApplied.
100 return MediaStreamVideoTrack::CreateVideoTrack( 99 return MediaStreamVideoTrack::CreateVideoTrack(
(...skipping 194 matching lines...) Expand 10 before | Expand all | Expand 10 after
295 std::unique_ptr<MockVideoCapturerSource> delegate( 294 std::unique_ptr<MockVideoCapturerSource> delegate(
296 new MockVideoCapturerSource()); 295 new MockVideoCapturerSource());
297 delegate_ = delegate.get(); 296 delegate_ = delegate.get();
298 source_ = new MediaStreamVideoCapturerSource( 297 source_ = new MediaStreamVideoCapturerSource(
299 base::Bind(&MediaStreamVideoCapturerSourceTest::OnSourceStopped, 298 base::Bind(&MediaStreamVideoCapturerSourceTest::OnSourceStopped,
300 base::Unretained(this)), 299 base::Unretained(this)),
301 std::move(delegate)); 300 std::move(delegate));
302 webkit_source_.initialize(base::UTF8ToUTF16("dummy_source_id"), 301 webkit_source_.initialize(base::UTF8ToUTF16("dummy_source_id"),
303 blink::WebMediaStreamSource::TypeVideo, 302 blink::WebMediaStreamSource::TypeVideo,
304 base::UTF8ToUTF16("dummy_source_name"), 303 base::UTF8ToUTF16("dummy_source_name"),
305 false /* remote */, true /* readonly */); 304 false /* remote */);
306 webkit_source_.setExtraData(source_); 305 webkit_source_.setExtraData(source_);
307 webkit_source_id_ = webkit_source_.id(); 306 webkit_source_id_ = webkit_source_.id();
308 307
309 InSequence s; 308 InSequence s;
310 EXPECT_CALL(mock_delegate(), GetCurrentSupportedFormats(_, _, _, _)); 309 EXPECT_CALL(mock_delegate(), GetCurrentSupportedFormats(_, _, _, _));
311 EXPECT_CALL(mock_delegate(), StartCapture(_, _, _)); 310 EXPECT_CALL(mock_delegate(), StartCapture(_, _, _));
312 blink::WebMediaStreamTrack track = StartSource(); 311 blink::WebMediaStreamTrack track = StartSource();
313 message_loop_.RunUntilIdle(); 312 message_loop_.RunUntilIdle();
314 313
315 OnStarted(true); 314 OnStarted(true);
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
399 run_loop.Run(); 398 run_loop.Run();
400 fake_sink.DisconnectFromTrack(); 399 fake_sink.DisconnectFromTrack();
401 EXPECT_EQ(reference_capture_time, capture_time); 400 EXPECT_EQ(reference_capture_time, capture_time);
402 double metadata_value; 401 double metadata_value;
403 EXPECT_TRUE(metadata.GetDouble(media::VideoFrameMetadata::FRAME_RATE, 402 EXPECT_TRUE(metadata.GetDouble(media::VideoFrameMetadata::FRAME_RATE,
404 &metadata_value)); 403 &metadata_value));
405 EXPECT_EQ(30.0, metadata_value); 404 EXPECT_EQ(30.0, metadata_value);
406 } 405 }
407 406
408 } // namespace content 407 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/media/canvas_capture_handler.cc ('k') | content/renderer/media/media_stream_video_source_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698