Bug 1273965: refactor to remove NS_NOTREACHED
MozReview-Commit-ID: 6xmDGv6oZUr
--- a/media/webrtc/signaling/src/peerconnection/MediaPipelineFactory.cpp
+++ b/media/webrtc/signaling/src/peerconnection/MediaPipelineFactory.cpp
@@ -993,31 +993,29 @@ MediaPipelineFactory::EnsureExternalCode
#else
encoder = GmpVideoCodec::CreateEncoder();
#endif
if (encoder) {
return aConduit.SetExternalSendCodec(aConfig, encoder);
} else {
return kMediaConduitInvalidSendCodec;
}
- } else {
- VideoDecoder* decoder = nullptr;
+ }
+ VideoDecoder* decoder = nullptr;
#ifdef MOZ_WEBRTC_OMX
- decoder =
- OMXVideoCodec::CreateDecoder(OMXVideoCodec::CodecType::CODEC_H264);
+ decoder =
+ OMXVideoCodec::CreateDecoder(OMXVideoCodec::CodecType::CODEC_H264);
#else
- decoder = GmpVideoCodec::CreateDecoder();
+ decoder = GmpVideoCodec::CreateDecoder();
#endif
- if (decoder) {
- return aConduit.SetExternalRecvCodec(aConfig, decoder);
- } else {
- return kMediaConduitInvalidReceiveCodec;
- }
+ if (decoder) {
+ return aConduit.SetExternalRecvCodec(aConfig, decoder);
+ } else {
+ return kMediaConduitInvalidReceiveCodec;
}
- NS_NOTREACHED("Shouldn't get here!");
} else {
MOZ_MTLOG(ML_ERROR,
"Invalid video codec configured: " << aConfig->mName.c_str());
return aIsSend ? kMediaConduitInvalidSendCodec
: kMediaConduitInvalidReceiveCodec;
}
NS_NOTREACHED("Shouldn't get here!");