Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(688)

Unified Diff: content/renderer/media/webrtc/rtc_stats.cc

Issue 2395503002: WebRTCStatsMember: Support for bool and sequence of bool (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/media/webrtc/rtc_stats.cc
diff --git a/content/renderer/media/webrtc/rtc_stats.cc b/content/renderer/media/webrtc/rtc_stats.cc
index 9d0a3bf5d8486b45744ba21f22359c644d4fbeaa..06aa1c79362248290dde8f04fde6232e0f02dba8 100644
--- a/content/renderer/media/webrtc/rtc_stats.cc
+++ b/content/renderer/media/webrtc/rtc_stats.cc
@@ -98,6 +98,8 @@ blink::WebString RTCStatsMember::name() const {
blink::WebRTCStatsMemberType RTCStatsMember::type() const {
switch (member_->type()) {
+ case webrtc::RTCStatsMemberInterface::kBool:
+ return blink::WebRTCStatsMemberTypeBool;
case webrtc::RTCStatsMemberInterface::kInt32:
return blink::WebRTCStatsMemberTypeInt32;
case webrtc::RTCStatsMemberInterface::kUint32:
@@ -110,6 +112,8 @@ blink::WebRTCStatsMemberType RTCStatsMember::type() const {
return blink::WebRTCStatsMemberTypeDouble;
case webrtc::RTCStatsMemberInterface::kString:
return blink::WebRTCStatsMemberTypeString;
+ case webrtc::RTCStatsMemberInterface::kSequenceBool:
+ return blink::WebRTCStatsMemberTypeSequenceBool;
case webrtc::RTCStatsMemberInterface::kSequenceInt32:
return blink::WebRTCStatsMemberTypeSequenceInt32;
case webrtc::RTCStatsMemberInterface::kSequenceUint32:
@@ -132,6 +136,11 @@ bool RTCStatsMember::isDefined() const {
return member_->is_defined();
}
+bool RTCStatsMember::valueBool() const {
+ DCHECK(isDefined());
+ return *member_->cast_to<webrtc::RTCStatsMember<bool>>();
+}
+
int32_t RTCStatsMember::valueInt32() const {
DCHECK(isDefined());
return *member_->cast_to<webrtc::RTCStatsMember<int32_t>>();
@@ -163,6 +172,18 @@ blink::WebString RTCStatsMember::valueString() const {
*member_->cast_to<webrtc::RTCStatsMember<std::string>>());
}
+blink::WebVector<int> RTCStatsMember::valueSequenceBool() const {
+ DCHECK(isDefined());
+ const std::vector<bool>& vector =
+ *member_->cast_to<webrtc::RTCStatsMember<std::vector<bool>>>();
+ std::vector<int> uint32_vector;
+ uint32_vector.reserve(vector.size());
+ for (size_t i = 0; i < vector.size(); ++i) {
+ uint32_vector.push_back(vector[i] ? 1 : 0);
+ }
+ return blink::WebVector<int>(uint32_vector);
+}
+
blink::WebVector<int32_t> RTCStatsMember::valueSequenceInt32() const {
DCHECK(isDefined());
return blink::WebVector<int32_t>(

Powered by Google App Engine
This is Rietveld 408576698