Fixes for memory and threading issues in cast

media/cast/video_sender/video_sender.cc

Index: media/cast/video_sender/video_sender.cc
diff --git a/media/cast/video_sender/video_sender.cc b/media/cast/video_sender/video_sender.cc
index 16f8f6b5fbac8adeef7bce2ed81c8b9c734b7476..a679fa305a1f18ab4db51ce17d914a51648e0f6c 100644
--- a/media/cast/video_sender/video_sender.cc
+++ b/media/cast/video_sender/video_sender.cc
@@ -185,12 +185,14 @@ void VideoSender::SendRtcpReport() {
       cast_environment_->Logging()->GetVideoRtcpRawData();
 
   while (!video_logs.empty()) {
+    // TODO(hclam): Avoid calling begin() within a loop.
     VideoRtcpRawMap::iterator it = video_logs.begin();
     uint32 rtp_timestamp = it->first;
 
     transport::RtcpSenderFrameLogMessage frame_message;
     frame_message.rtp_timestamp = rtp_timestamp;
     frame_message.frame_status = transport::kRtcpSenderFrameStatusUnknown;
+    bool ignore_event = false;
 
     switch (it->second.type) {
       case kVideoFrameCaptured:
@@ -206,11 +208,11 @@ void VideoSender::SendRtcpReport() {
             transport::kRtcpSenderFrameStatusSentToNetwork;
         break;
       default:
-        NOTREACHED();
-        break;
+        ignore_event = true;
     }
     video_logs.erase(rtp_timestamp);
-    sender_log_message.push_back(frame_message);
+    if (!ignore_event)
+      sender_log_message.push_back(frame_message);
   }
 
   rtcp_->SendRtcpFromRtpSender(sender_log_message);