Remove |remote| and |readonly| members of MediaStreamTrack

content/renderer/media/external_media_stream_audio_source.cc

Index: content/renderer/media/external_media_stream_audio_source.cc
diff --git a/content/renderer/media/external_media_stream_audio_source.cc b/content/renderer/media/external_media_stream_audio_source.cc
index 60a68241b6fff060060078e60693b6ea43010041..d894f21b058149aab04d6a2d82d48c45f41af13b 100644
--- a/content/renderer/media/external_media_stream_audio_source.cc
+++ b/content/renderer/media/external_media_stream_audio_source.cc
@@ -10,9 +10,9 @@ ExternalMediaStreamAudioSource::ExternalMediaStreamAudioSource(
     scoped_refptr<media::AudioCapturerSource> source,
     int sample_rate,
     media::ChannelLayout channel_layout,
-    int frames_per_buffer,
-    bool is_remote)
-    : MediaStreamAudioSource(!is_remote), source_(std::move(source)),
+    int frames_per_buffer)
+    : MediaStreamAudioSource(true),

[mcasas, 2016/10/17 18:08:25]

I would say that this parameter ripples through to
MediaStreamAudioSource::is_local_source_ [1], which
is probably made superfluous by this CL, and should
be removed. If that's not done in this CL, please add
a TODO with indication of the bug in that class.

(Note that MediaStreamAudioSource::is_local_source_ 
is  only used for a DVLOG, so probably is safe to 
remove right away).

https://cs.chromium.org/chromium/src/content/renderer/media/media_stream_audi...

+      source_(std::move(source)),
       was_started_(false) {
   DVLOG(1)
       << "ExternalMediaStreamAudioSource::ExternalMediaStreamAudioSource()";