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

Unified Diff: media/cast/logging/logging_defines.cc

Issue 236123003: Cast: Provide more meaningful stats. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: minor fix Created 6 years, 8 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: media/cast/logging/logging_defines.cc
diff --git a/media/cast/logging/logging_defines.cc b/media/cast/logging/logging_defines.cc
index c00c420031afc0bd0e52ef392e89a98f6cad7fda..e56d43c51484128cc191973f3cccd4c055422bb4 100644
--- a/media/cast/logging/logging_defines.cc
+++ b/media/cast/logging/logging_defines.cc
@@ -46,6 +46,8 @@ const char* CastLoggingToString(CastLoggingEvent event) {
ENUM_TO_STRING(VideoPacketReceived);
ENUM_TO_STRING(DuplicateAudioPacketReceived);
ENUM_TO_STRING(DuplicateVideoPacketReceived);
+ ENUM_TO_STRING(AudioRtcpPacketReceived);
+ ENUM_TO_STRING(VideoRtcpPacketReceived);
}
NOTREACHED();
return "";
@@ -70,6 +72,7 @@ EventMediaType GetEventMediaType(CastLoggingEvent event) {
case kAudioPacketRetransmitted:
case kAudioPacketReceived:
case kDuplicateAudioPacketReceived:
+ case kAudioRtcpPacketReceived:
return AUDIO_EVENT;
case kVideoAckReceived:
case kVideoAckSent:
@@ -84,6 +87,7 @@ EventMediaType GetEventMediaType(CastLoggingEvent event) {
case kVideoPacketRetransmitted:
case kVideoPacketReceived:
case kDuplicateVideoPacketReceived:
+ case kVideoRtcpPacketReceived:
return VIDEO_EVENT;
}
NOTREACHED();
@@ -106,23 +110,5 @@ PacketEvent::~PacketEvent() {}
GenericEvent::GenericEvent() : type(kUnknown), value(0) {}
GenericEvent::~GenericEvent() {}
-
-FrameLogStats::FrameLogStats()
- : event_counter(0),
- sum_size(0) {}
-FrameLogStats::~FrameLogStats() {}
-
-PacketLogStats::PacketLogStats()
- : event_counter(0),
- sum_size(0) {}
-PacketLogStats::~PacketLogStats() {}
-
-GenericLogStats::GenericLogStats()
- : event_counter(0),
- sum(0),
- sum_squared(0),
- min(0),
- max(0) {}
-GenericLogStats::~GenericLogStats() {}
} // namespace cast
} // namespace media

Powered by Google App Engine
This is Rietveld 408576698