Ignore invalid optional constraints with invalid aspect ratio.

content/renderer/media/media_stream_video_source.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/media/media_stream_video_source.h"
 
 #include <algorithm>
 #include <limits>
 #include <string>
 
@@ skipping 88 matching lines @@
   }
   // Note - the code below will potentially pick min and max from different
   // constraint sets, some of which might have been ignored.
   for (const auto& constraint_set : constraints.advanced()) {
     if (constraint_set.aspectRatio.hasMin()) {
       *min_aspect_ratio = constraint_set.aspectRatio.min();
       break;
     }
   }
   for (const auto& constraint_set : constraints.advanced()) {
-    if (constraint_set.aspectRatio.hasMax()) {
+    // Advanced constraint sets with max aspect ratio 0 are unsatisfiable and
+    // must be ignored.
+    if (constraint_set.aspectRatio.hasMax() &&
+        constraint_set.aspectRatio.max() > 0) {
       *max_aspect_ratio = constraint_set.aspectRatio.max();
       break;
     }
   }
 }
 
 // Returns true if |constraints| are fulfilled. |format| can be changed by a
 // constraint, e.g. the frame rate can be changed by setting maxFrameRate.
 bool UpdateFormatForConstraints(
     const blink::WebMediaTrackConstraintSet& constraints,
@@ skipping 495 matching lines @@
       callback(callback) {
 }
 
 MediaStreamVideoSource::TrackDescriptor::TrackDescriptor(
     const TrackDescriptor& other) = default;
 
 MediaStreamVideoSource::TrackDescriptor::~TrackDescriptor() {
 }
 
 }  // namespace content