diff --git a/sdk/android/src/jni/pc/rtc_stats_collector_callback_wrapper.cc b/sdk/android/src/jni/pc/rtc_stats_collector_callback_wrapper.cc index 8d0e34c575..dc1dcf382b 100644 --- a/sdk/android/src/jni/pc/rtc_stats_collector_callback_wrapper.cc +++ b/sdk/android/src/jni/pc/rtc_stats_collector_callback_wrapper.cc @@ -42,80 +42,53 @@ ScopedJavaLocalRef NativeToJavaBigIntegerArray( env, container, java_math_BigInteger_clazz(env), &NativeToJavaBigInteger); } -ScopedJavaLocalRef MemberToJava( - JNIEnv* env, - const RTCStatsMemberInterface& member) { - switch (member.type()) { - case RTCStatsMemberInterface::kBool: - return NativeToJavaBoolean(env, *member.cast_to>()); - - case RTCStatsMemberInterface::kInt32: - return NativeToJavaInteger(env, - *member.cast_to>()); - - case RTCStatsMemberInterface::kUint32: - return NativeToJavaLong(env, *member.cast_to>()); - - case RTCStatsMemberInterface::kInt64: - return NativeToJavaLong(env, *member.cast_to>()); - - case RTCStatsMemberInterface::kUint64: - return NativeToJavaBigInteger( - env, *member.cast_to>()); - - case RTCStatsMemberInterface::kDouble: - return NativeToJavaDouble(env, *member.cast_to>()); - - case RTCStatsMemberInterface::kString: - return NativeToJavaString(env, - *member.cast_to>()); - - case RTCStatsMemberInterface::kSequenceBool: - return NativeToJavaBooleanArray( - env, *member.cast_to>>()); - - case RTCStatsMemberInterface::kSequenceInt32: - return NativeToJavaIntegerArray( - env, *member.cast_to>>()); - - case RTCStatsMemberInterface::kSequenceUint32: { - const std::vector& v = - *member.cast_to>>(); - return NativeToJavaLongArray(env, - std::vector(v.begin(), v.end())); - } - case RTCStatsMemberInterface::kSequenceInt64: - return NativeToJavaLongArray( - env, *member.cast_to>>()); - - case RTCStatsMemberInterface::kSequenceUint64: - return NativeToJavaBigIntegerArray( - env, *member.cast_to>>()); - - case RTCStatsMemberInterface::kSequenceDouble: - return NativeToJavaDoubleArray( - env, *member.cast_to>>()); - - case RTCStatsMemberInterface::kSequenceString: - return NativeToJavaStringArray( - env, *member.cast_to>>()); - - case RTCStatsMemberInterface::kMapStringUint64: - return NativeToJavaMap( - env, - *member.cast_to>>(), - [](JNIEnv* env, const auto& entry) { - return std::make_pair(NativeToJavaString(env, entry.first), - NativeToJavaBigInteger(env, entry.second)); - }); - - case RTCStatsMemberInterface::kMapStringDouble: - return NativeToJavaMap( - env, *member.cast_to>>(), - [](JNIEnv* env, const auto& entry) { - return std::make_pair(NativeToJavaString(env, entry.first), - NativeToJavaDouble(env, entry.second)); - }); +ScopedJavaLocalRef AttributeToJava(JNIEnv* env, + const Attribute& attribute) { + if (attribute.holds_alternative()) { + return NativeToJavaBoolean(env, attribute.get()); + } else if (attribute.holds_alternative()) { + return NativeToJavaInteger(env, attribute.get()); + } else if (attribute.holds_alternative()) { + return NativeToJavaLong(env, attribute.get()); + } else if (attribute.holds_alternative()) { + return NativeToJavaLong(env, attribute.get()); + } else if (attribute.holds_alternative()) { + return NativeToJavaBigInteger(env, attribute.get()); + } else if (attribute.holds_alternative()) { + return NativeToJavaDouble(env, attribute.get()); + } else if (attribute.holds_alternative()) { + return NativeToJavaString(env, attribute.get()); + } else if (attribute.holds_alternative>()) { + return NativeToJavaBooleanArray(env, attribute.get>()); + } else if (attribute.holds_alternative>()) { + return NativeToJavaIntegerArray(env, attribute.get>()); + } else if (attribute.holds_alternative>()) { + const std::vector& v = attribute.get>(); + return NativeToJavaLongArray(env, std::vector(v.begin(), v.end())); + } else if (attribute.holds_alternative>()) { + return NativeToJavaLongArray(env, attribute.get>()); + } else if (attribute.holds_alternative>()) { + return NativeToJavaBigIntegerArray(env, + attribute.get>()); + } else if (attribute.holds_alternative>()) { + return NativeToJavaDoubleArray(env, attribute.get>()); + } else if (attribute.holds_alternative>()) { + return NativeToJavaStringArray(env, + attribute.get>()); + } else if (attribute.holds_alternative>()) { + return NativeToJavaMap( + env, attribute.get>(), + [](JNIEnv* env, const auto& entry) { + return std::make_pair(NativeToJavaString(env, entry.first), + NativeToJavaBigInteger(env, entry.second)); + }); + } else if (attribute.holds_alternative>()) { + return NativeToJavaMap(env, attribute.get>(), + [](JNIEnv* env, const auto& entry) { + return std::make_pair( + NativeToJavaString(env, entry.first), + NativeToJavaDouble(env, entry.second)); + }); } RTC_DCHECK_NOTREACHED(); return nullptr; @@ -124,11 +97,11 @@ ScopedJavaLocalRef MemberToJava( ScopedJavaLocalRef NativeToJavaRtcStats(JNIEnv* env, const RTCStats& stats) { JavaMapBuilder builder(env); - for (auto* const member : stats.Members()) { - if (!member->is_defined()) + for (const auto& attribute : stats.Attributes()) { + if (!attribute.has_value()) continue; - builder.put(NativeToJavaString(env, member->name()), - MemberToJava(env, *member)); + builder.put(NativeToJavaString(env, attribute.name()), + AttributeToJava(env, attribute)); } return Java_RTCStats_create( env, stats.timestamp().us(), NativeToJavaString(env, stats.type()),