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

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

Issue 69603002: Incorporating logging into Cast (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: clean up Created 7 years, 1 month 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_raw.cc
diff --git a/media/cast/logging/logging_raw.cc b/media/cast/logging/logging_raw.cc
index fa865a432433593cb9d0fb4458f0feecfb86e55a..718cf6986cb3500f1079f29aa53085c9c748bc9b 100644
--- a/media/cast/logging/logging_raw.cc
+++ b/media/cast/logging/logging_raw.cc
@@ -64,6 +64,12 @@ void LoggingRaw::InsertBaseFrameEvent(CastLoggingEvent event,
// Insert to an existing entry.
it->second.timestamp.push_back(clock_->NowTicks());
it->second.type.push_back(event);
+ // Do we have a valid frame_id?
+ // We don't always have it to begin with.
+ // TODO(mikhal): Switch frame_id to int when the fix gets in.
+ // This is currently illegal, as frame_id is uint8.
+ if (it->second.frame_id == -1 && frame_id != -1)
+ it->second.frame_id = frame_id;
}
}
@@ -72,7 +78,7 @@ void LoggingRaw::InsertPacketEvent(CastLoggingEvent event,
uint8 frame_id,
uint16 packet_id,
uint16 max_packet_id,
- int size) {
+ size_t size) {
// Is this packet belonging to a new frame?
PacketRawMap::iterator it = packet_map_.find(rtp_timestamp);
if (it == packet_map_.end()) {

Powered by Google App Engine
This is Rietveld 408576698