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),

[miu, 2016/10/23 00:51:30]

The value for |is_remote| was true here (from the
cast_streaming_native_handler.cc), which would call the base class ctor with
false. This will mess-up the UMA histograms in content::TrackAudioRenderer
(e.g.,
https://cs.chromium.org/chromium/src/content/renderer/media/track_audio_rende...).

Is that acceptable? If you haven't already, please contact the owner of those
UMAs.

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