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

Side by Side Diff: components/test_runner/mock_web_user_media_client.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
« no previous file with comments | « no previous file | components/test_runner/mock_webrtc_peer_connection_handler.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 "components/test_runner/mock_web_user_media_client.h" 5 #include "components/test_runner/mock_web_user_media_client.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/bind_helpers.h" 10 #include "base/bind_helpers.h"
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
62 stream.initialize(WebVector<WebMediaStreamTrack>(), 62 stream.initialize(WebVector<WebMediaStreamTrack>(),
63 WebVector<WebMediaStreamTrack>()); 63 WebVector<WebMediaStreamTrack>());
64 stream.setExtraData(new MockExtraData()); 64 stream.setExtraData(new MockExtraData());
65 65
66 if (request.audio() && 66 if (request.audio() &&
67 !delegate_->AddMediaStreamAudioSourceAndTrack(&stream)) { 67 !delegate_->AddMediaStreamAudioSourceAndTrack(&stream)) {
68 WebMediaStreamSource source; 68 WebMediaStreamSource source;
69 source.initialize("MockAudioDevice#1", 69 source.initialize("MockAudioDevice#1",
70 WebMediaStreamSource::TypeAudio, 70 WebMediaStreamSource::TypeAudio,
71 "Mock audio device", 71 "Mock audio device",
72 false /* remote */, 72 false /* remote */);
73 true /* readonly */);
74 WebMediaStreamTrack web_track; 73 WebMediaStreamTrack web_track;
75 web_track.initialize(source); 74 web_track.initialize(source);
76 stream.addTrack(web_track); 75 stream.addTrack(web_track);
77 } 76 }
78 77
79 if (request.video() && 78 if (request.video() &&
80 !delegate_->AddMediaStreamVideoSourceAndTrack(&stream)) { 79 !delegate_->AddMediaStreamVideoSourceAndTrack(&stream)) {
81 WebMediaStreamSource source; 80 WebMediaStreamSource source;
82 source.initialize("MockVideoDevice#1", 81 source.initialize("MockVideoDevice#1",
83 WebMediaStreamSource::TypeVideo, 82 WebMediaStreamSource::TypeVideo,
84 "Mock video device", 83 "Mock video device",
85 false /* remote */, true /* readonly */); 84 false /* remote */);
86 WebMediaStreamTrack web_track; 85 WebMediaStreamTrack web_track;
87 web_track.initialize(source); 86 web_track.initialize(source);
88 stream.addTrack(web_track); 87 stream.addTrack(web_track);
89 } 88 }
90 89
91 delegate_->PostTask(new WebCallbackTask( 90 delegate_->PostTask(new WebCallbackTask(
92 base::Bind(&WebUserMediaRequest::requestSucceeded, 91 base::Bind(&WebUserMediaRequest::requestSucceeded,
93 base::Owned(new WebUserMediaRequest(request)), stream))); 92 base::Owned(new WebUserMediaRequest(request)), stream)));
94 } 93 }
95 94
(...skipping 75 matching lines...) Expand 10 before | Expand all | Expand 10 after
171 WebString::fromUTF8(test_sources[i].label), 170 WebString::fromUTF8(test_sources[i].label),
172 test_sources[i].facing); 171 test_sources[i].facing);
173 } 172 }
174 173
175 delegate_->PostTask(new WebCallbackTask(base::Bind( 174 delegate_->PostTask(new WebCallbackTask(base::Bind(
176 &WebMediaStreamTrackSourcesRequest::requestSucceeded, 175 &WebMediaStreamTrackSourcesRequest::requestSucceeded,
177 base::Owned(new WebMediaStreamTrackSourcesRequest(request)), sources))); 176 base::Owned(new WebMediaStreamTrackSourcesRequest(request)), sources)));
178 } 177 }
179 178
180 } // namespace test_runner 179 } // namespace test_runner
OLDNEW
« no previous file with comments | « no previous file | components/test_runner/mock_webrtc_peer_connection_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698