AndroidVideoCapturer: Delegate framerate choice to VideoCapturerAndroid.java

talk/app/webrtc/videosource.cc

Index: talk/app/webrtc/videosource.cc
diff --git a/talk/app/webrtc/videosource.cc b/talk/app/webrtc/videosource.cc
index d6e92b0f33385437da37d8f9a53e8f4073c7290d..bb27b10bcdf54a56b2f131330ae640611c925af2 100644
--- a/talk/app/webrtc/videosource.cc
+++ b/talk/app/webrtc/videosource.cc
@@ -157,12 +157,9 @@ bool NewFormatWithConstraints(
         value = 1;
       }
     }
-    if (value <= cricket::VideoFormat::IntervalToFps(format_in.interval)) {
+    if (value <= cricket::VideoFormat::IntervalToFps(format_in.interval))
       format_out->interval = cricket::VideoFormat::FpsToInterval(value);
-      return true;
-    } else {
-      return false;

[magjed_webrtc, 2015/08/27 13:50:23]

I removed this because it looks like a bug to me. We should not filter out
formats where supportedFps < constraint.kMaxFrameRate.

-    }
+    return true;
   } else if (constraint.key == MediaConstraintsInterface::kMinAspectRatio) {
     double value = rtc::FromString<double>(constraint.value);
     // The aspect ratio in |constraint.value| has been converted to a string and