Bug 1166183 - Reset PipelineListener's flag after ReplaceTrack(). r=bwc, a=sledru
authorAndreas Pehrson <pehrsons@gmail.com>
Thu, 21 May 2015 13:35:29 +0800
changeset 266107 d68fc8a46c1a
parent 266106 7311bd491c30
child 266108 46bb07934327
push id4755
push userryanvm@gmail.com
push date2015-05-25 19:46 +0000
treeherdermozilla-beta@46bb07934327 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbwc, sledru
bugs1166183
milestone39.0
Bug 1166183 - Reset PipelineListener's flag after ReplaceTrack(). r=bwc, a=sledru
media/webrtc/signaling/src/mediapipeline/MediaPipeline.cpp
--- a/media/webrtc/signaling/src/mediapipeline/MediaPipeline.cpp
+++ b/media/webrtc/signaling/src/mediapipeline/MediaPipeline.cpp
@@ -625,22 +625,20 @@ void MediaPipelineTransmit::AttachToTrac
 
   // TODO(ekr@rtfm.com): Check for errors
   MOZ_MTLOG(ML_DEBUG, "Attaching pipeline to stream "
             << static_cast<void *>(stream_) << " conduit type=" <<
             (conduit_->type() == MediaSessionConduit::AUDIO ?"audio":"video"));
 
   stream_->AddListener(listener_);
 
- // Is this a gUM mediastream?  If so, also register the Listener directly with
- // the SourceMediaStream that's attached to the TrackUnion so we can get direct
- // unqueued (and not resampled) data
- if (domstream_->AddDirectListener(listener_)) {
-   listener_->direct_connect_ = true;
- }
+  // Is this a gUM mediastream?  If so, also register the Listener directly with
+  // the SourceMediaStream that's attached to the TrackUnion so we can get direct
+  // unqueued (and not resampled) data
+  listener_->direct_connect_ = domstream_->AddDirectListener(listener_);
 
 #ifndef MOZILLA_INTERNAL_API
   // this enables the unit tests that can't fiddle with principals and the like
   listener_->SetEnabled(true);
 #endif
 }
 
 #ifdef MOZILLA_INTERNAL_API