diff --git a/video/video_receive_stream.cc b/video/video_receive_stream.cc index 2d99be690c..c495f4e5b9 100644 --- a/video/video_receive_stream.cc +++ b/video/video_receive_stream.cc @@ -348,9 +348,13 @@ void VideoReceiveStream::Start() { std::string decoded_output_file = field_trial::FindFullName("WebRTC-DecoderDataDumpDirectory"); - // Because '/' can't be used inside a field trial parameter, we use ':' + // Because '/' can't be used inside a field trial parameter, we use ';' // instead. - absl::c_replace(decoded_output_file, ':', '/'); + // This is only relevant to WebRTC-DecoderDataDumpDirectory + // field trial. ';' is chosen arbitrary. Even though it's a legal character + // in some file systems, we can sacrifice ability to use it in the path to + // dumped video, since it's developers-only feature for debugging. + absl::c_replace(decoded_output_file, ';', '/'); if (!decoded_output_file.empty()) { char filename_buffer[256]; rtc::SimpleStringBuilder ssb(filename_buffer);