Added the JNI interface to get and set RtpParameters and the maximum bitrate limits.

webrtc/api/java/jni/jni_helpers.cc

 /*
  *  Copyright 2015 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.
  */
 #include "webrtc/api/java/jni/jni_helpers.h"
@@ skipping 254 matching lines @@
 // Scope Java local references to the lifetime of this object.  Use in all C++
 // callbacks (i.e. entry points that don't originate in a Java callstack
 // through a "native" method call).
 ScopedLocalRefFrame::ScopedLocalRefFrame(JNIEnv* jni) : jni_(jni) {
   RTC_CHECK(!jni_->PushLocalFrame(0)) << "Failed to PushLocalFrame";
 }
 ScopedLocalRefFrame::~ScopedLocalRefFrame() {
   jni_->PopLocalFrame(NULL);
 }
 
+// Creates an iterator representing the end of any collection.
+Iterator::Iterator() : iterator_(NULL) {}

[magjed_webrtc, 2016/03/22 15:15:30]

nit: Use nullptr instead of NULL. This file seems to use both NULL and nullptr,
but we should probably try to use nullptr.

[skvlad, 2016/03/23 00:29:27]

Thanks. I've updated the entire file to use nullptr.

+
+// Creates an iterator pointing to the beginning of the specified collection.
+Iterator::Iterator(JNIEnv* jni, jobject iterable) : jni_(jni) {
+  jclass j_class = GetObjectClass(jni, iterable);
+  jmethodID iterator_id =
+      GetMethodID(jni, j_class, "iterator", "()Ljava/util/Iterator;");
+  iterator_ = jni->CallObjectMethod(iterable, iterator_id);
+  CHECK_EXCEPTION(jni) << "error during CallObjectMethod";
+  RTC_CHECK(iterator_ != NULL);
+
+  jclass iterator_class = GetObjectClass(jni, iterator_);
+  has_next_id_ = GetMethodID(jni, iterator_class, "hasNext", "()Z");
+  next_id_ = GetMethodID(jni, iterator_class, "next", "()Ljava/lang/Object;");
+
+  // Start at the first element in the collection.
+  ++(*this);
+}
+
+// Move constructor - necessary to be able to return iterator types from
+// functions.
+Iterator::Iterator(Iterator&& other)
+    : jni_(std::move(other.jni_)),
+      iterator_(std::move(other.iterator_)),
+      value_(std::move(other.value_)),
+      has_next_id_(std::move(other.has_next_id_)),
+      next_id_(std::move(other.next_id_)){};

[magjed_webrtc, 2016/03/22 15:15:30]

nit: space between '(' and '{'. Can you use 'git cl format' on this file?

[skvlad, 2016/03/23 00:29:27]

Done. Curiously, "git cl format" wants to remove this space when I add it.

+
+// Advances the iterator one step.
+Iterator& Iterator::operator++() {
+  if (iterator_ == NULL) {
+    // Can't iterate past the end.
+    return *this;
+  }
+  bool has_next = jni_->CallBooleanMethod(iterator_, has_next_id_);
+  CHECK_EXCEPTION(jni_) << "error during CallBooleanMethod";
+  if (!has_next) {

[magjed_webrtc, 2016/03/22 15:15:30]

You should probably set |value_| to nullptr here as well.

[skvlad, 2016/03/23 00:29:26]

Done.

+    iterator_ = NULL;
+    return *this;
+  }
+
+  value_ = jni_->CallObjectMethod(iterator_, next_id_);
+  CHECK_EXCEPTION(jni_) << "error during CallObjectMethod";
+  return *this;
+}
+
 }  // namespace webrtc_jni