Relax parsing of a=bundle-only with a nonzero port.

webrtc/api/webrtcsdp.cc

 /*
  *  Copyright 2011 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 2354 matching lines @@
     } else {
       LOG(LS_WARNING) << "Unsupported media type: " << line;
       continue;
     }
     if (!content.get()) {
       // ParseContentDescription returns NULL if failed.
       return false;
     }
 
     bool content_rejected = false;
+    // A port of 0 is not interpreted as a rejected m= section when it's
+    // used along with a=bundle-only.
     if (bundle_only) {
-      // A port of 0 is not interpreted as a rejected m= section when it's
-      // used along with a=bundle-only.
       if (!port_rejected) {
-        return ParseFailed(
-            "a=bundle-only",
-            "a=bundle-only MUST only be used in combination with a 0 port.",
-            error);
+        // Usage of bundle-only with a nonzero port is unspecified. So just
+        // ignore bundle-only if we see this.
+        bundle_only = false;
+        LOG(LS_WARNING)
+            << "a=bundle-only attribute observed with a nonzero "
+            << "port; this usage is unspecified so the attribute is being "
+            << "ignored.";
       }
     } else {
       // If not using bundle-only, interpret port 0 in the normal way; the m=
       // section is being rejected.
       content_rejected = port_rejected;
     }
 
     if (IsRtp(protocol)) {
       // Set the extmap.
       if (!session_extmaps.empty() &&
@@ skipping 828 matching lines @@
     UpdateCodec<AudioContentDescription, cricket::AudioCodec>(
         media_desc, payload_type, feedback_param);
   } else if (media_type == cricket::MEDIA_TYPE_VIDEO) {
     UpdateCodec<VideoContentDescription, cricket::VideoCodec>(
         media_desc, payload_type, feedback_param);
   }
   return true;
 }
 
 }  // namespace webrtc