Remove |remote| and |readonly| members of MediaStreamTrack

content/renderer/media/webmediaplayer_ms_compositor.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/media/webmediaplayer_ms_compositor.h"
 
 #include <stdint.h>
 #include <string>
 
 #include "base/command_line.h"
@@ skipping 124 matching lines @@
       total_frame_count_(0),
       dropped_frame_count_(0),
       stopped_(true),
       weak_ptr_factory_(this) {
   main_message_loop_ = base::MessageLoop::current();
 
   blink::WebVector<blink::WebMediaStreamTrack> video_tracks;
   if (!web_stream.isNull())
     web_stream.videoTracks(video_tracks);
 
-  const bool remote_video =
-      video_tracks.size() && video_tracks[0].source().remote();
+  const bool remote_video = video_tracks.size();
 
   if (remote_video &&

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

l.145-146 in the LHS tested video_tracks.size() as a 
way to avoid crashing when accessing video_tracks[0];
with that and since now we're assuming |remote| is 
always false, I think we can just remove |remote_video| 
and the whole pack of lines l.147-155 in the RHS, and 
simplify l. 160, wdyt?

       !base::CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kDisableRTCSmoothnessAlgorithm)) {
     base::AutoLock auto_lock(current_frame_lock_);
     rendering_frame_buffer_.reset(new media::VideoRendererAlgorithm(
         base::Bind(&WebMediaPlayerMSCompositor::MapTimestampsToRenderTimeTicks,
                    base::Unretained(this))));
   }
 
   // Just for logging purpose.
   std::string stream_id =
@@ skipping 279 matching lines @@
   }
 
   if (!rendering_frame_buffer_) {
     rendering_frame_buffer_.reset(new media::VideoRendererAlgorithm(
         base::Bind(&WebMediaPlayerMSCompositor::MapTimestampsToRenderTimeTicks,
                    base::Unretained(this))));
   }
 }
 
 }  // namespace content