Don't write spaces after semicolons in FMTP lines.
Reference: RFC 6184 section 8.2.1 and examples.
BUG=webrtc:5793
NOTRY=True
Review URL: https://codereview.webrtc.org/1888373002
Cr-Commit-Position: refs/heads/master@{#12383}
diff --git a/webrtc/api/webrtcsdp.cc b/webrtc/api/webrtcsdp.cc
index 8497f4c..10f9922 100644
--- a/webrtc/api/webrtcsdp.cc
+++ b/webrtc/api/webrtcsdp.cc
@@ -1564,11 +1564,13 @@
std::ostringstream* os) {
for (cricket::CodecParameterMap::const_iterator fmtp = parameters.begin();
fmtp != parameters.end(); ++fmtp) {
- // Each new parameter, except the first one starts with ";" and " ".
- if (fmtp != parameters.begin()) {
+ // Parameters are a semicolon-separated list, no spaces.
+ // The list is separated from the header by a space.
+ if (fmtp == parameters.begin()) {
+ *os << kSdpDelimiterSpace;
+ } else {
*os << kSdpDelimiterSemicolon;
}
- *os << kSdpDelimiterSpace;
WriteFmtpParameter(fmtp->first, fmtp->second, os);
}
}