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

Unified Diff: content/public/renderer/media_stream_utils.cc

Issue 2425703002: Remove |remote| and |readonly| members of MediaStreamTrack (Closed)
Patch Set: Remove |remote| and |readonly| members of MediaStreamTrack Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: content/public/renderer/media_stream_utils.cc
diff --git a/content/public/renderer/media_stream_utils.cc b/content/public/renderer/media_stream_utils.cc
index d05b08f5c86fe5281f613565d16aac927f6e71ae..70e99a95653978c57320c72c7a053181066be789 100644
--- a/content/public/renderer/media_stream_utils.cc
+++ b/content/public/renderer/media_stream_utils.cc
@@ -25,8 +25,6 @@ namespace content {
bool AddVideoTrackToMediaStream(
std::unique_ptr<media::VideoCapturerSource> video_source,
- bool is_remote,
- bool is_readonly,
blink::WebMediaStream* web_media_stream) {
DCHECK(video_source.get());
if (!web_media_stream || web_media_stream->isNull()) {
@@ -40,9 +38,8 @@ bool AddVideoTrackToMediaStream(
MediaStreamSource::SourceStoppedCallback(), std::move(video_source));
const blink::WebString track_id =
blink::WebString::fromUTF8(base::GenerateGUID());
- web_media_stream_source.initialize(track_id,
- blink::WebMediaStreamSource::TypeVideo,
- track_id, is_remote);
+ web_media_stream_source.initialize(
+ track_id, blink::WebMediaStreamSource::TypeVideo, track_id);
// Takes ownership of |media_stream_source|.
web_media_stream_source.setExtraData(media_stream_source);
@@ -59,8 +56,6 @@ bool AddAudioTrackToMediaStream(
int sample_rate,
media::ChannelLayout channel_layout,
int frames_per_buffer,
- bool is_remote,
- bool is_readonly,
blink::WebMediaStream* web_media_stream) {
DCHECK(audio_source.get());
if (!web_media_stream || web_media_stream->isNull()) {
@@ -79,13 +74,11 @@ bool AddAudioTrackToMediaStream(
blink::WebMediaStreamSource web_media_stream_source;
const blink::WebString track_id =
blink::WebString::fromUTF8(base::GenerateGUID());
- web_media_stream_source.initialize(track_id,
- blink::WebMediaStreamSource::TypeAudio,
- track_id, is_remote);
+ web_media_stream_source.initialize(
+ track_id, blink::WebMediaStreamSource::TypeAudio, track_id);
MediaStreamAudioSource* const media_stream_source =
new ExternalMediaStreamAudioSource(std::move(audio_source), sample_rate,
- channel_layout, frames_per_buffer,
- is_remote);
+ channel_layout, frames_per_buffer);
// Takes ownership of |media_stream_source|.
web_media_stream_source.setExtraData(media_stream_source);

Powered by Google App Engine
This is Rietveld 408576698