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

Side by Side Diff: media/cast/rtcp/rtcp_sender.cc

Issue 270493003: Cast: Deduplicate event types in cast library. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 6 years, 7 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 unified diff | Download patch
« no previous file with comments | « media/cast/rtcp/rtcp_receiver_unittest.cc ('k') | media/cast/rtcp/rtcp_sender_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "media/cast/rtcp/rtcp_sender.h" 5 #include "media/cast/rtcp/rtcp_sender.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 8
9 #include <algorithm> 9 #include <algorithm>
10 #include <vector> 10 #include <vector>
(...skipping 664 matching lines...) Expand 10 before | Expand all | Expand 10 after
675 675
676 while (!frame_log_messages.event_log_messages_.empty() && 676 while (!frame_log_messages.event_log_messages_.empty() &&
677 messages_in_frame > 0) { 677 messages_in_frame > 0) {
678 const RtcpReceiverEventLogMessage& event_message = 678 const RtcpReceiverEventLogMessage& event_message =
679 frame_log_messages.event_log_messages_.front(); 679 frame_log_messages.event_log_messages_.front();
680 uint16 event_type_and_timestamp_delta = 680 uint16 event_type_and_timestamp_delta =
681 MergeEventTypeAndTimestampForWireFormat( 681 MergeEventTypeAndTimestampForWireFormat(
682 event_message.type, 682 event_message.type,
683 event_message.event_timestamp - event_timestamp_base); 683 event_message.event_timestamp - event_timestamp_base);
684 switch (event_message.type) { 684 switch (event_message.type) {
685 case kAudioAckSent: 685 case FRAME_ACK_SENT:
686 case kVideoAckSent: 686 case FRAME_PLAYOUT:
687 case kAudioPlayoutDelay: 687 case FRAME_DECODED:
688 case kAudioFrameDecoded:
689 case kVideoFrameDecoded:
690 case kVideoRenderDelay:
691 big_endian_writer.WriteU16( 688 big_endian_writer.WriteU16(
692 static_cast<uint16>(event_message.delay_delta.InMilliseconds())); 689 static_cast<uint16>(event_message.delay_delta.InMilliseconds()));
693 big_endian_writer.WriteU16(event_type_and_timestamp_delta); 690 big_endian_writer.WriteU16(event_type_and_timestamp_delta);
694 break; 691 break;
695 case kAudioPacketReceived: 692 case PACKET_RECEIVED:
696 case kVideoPacketReceived:
697 case kDuplicateAudioPacketReceived:
698 case kDuplicateVideoPacketReceived:
699 big_endian_writer.WriteU16(event_message.packet_id); 693 big_endian_writer.WriteU16(event_message.packet_id);
700 big_endian_writer.WriteU16(event_type_and_timestamp_delta); 694 big_endian_writer.WriteU16(event_type_and_timestamp_delta);
701 break; 695 break;
702 default: 696 default:
703 NOTREACHED(); 697 NOTREACHED();
704 } 698 }
705 messages_in_frame--; 699 messages_in_frame--;
706 frame_log_messages.event_log_messages_.pop_front(); 700 frame_log_messages.event_log_messages_.pop_front();
707 } 701 }
708 if (frame_log_messages.event_log_messages_.empty()) { 702 if (frame_log_messages.event_log_messages_.empty()) {
(...skipping 113 matching lines...) Expand 10 before | Expand all | Expand 10 after
822 << "Not enough buffer space."; 816 << "Not enough buffer space.";
823 817
824 VLOG(3) << "number of frames: " << *number_of_frames; 818 VLOG(3) << "number of frames: " << *number_of_frames;
825 VLOG(3) << "total messages to send: " << *total_number_of_messages_to_send; 819 VLOG(3) << "total messages to send: " << *total_number_of_messages_to_send;
826 VLOG(3) << "rtcp log size: " << *rtcp_log_size; 820 VLOG(3) << "rtcp log size: " << *rtcp_log_size;
827 return *number_of_frames > 0; 821 return *number_of_frames > 0;
828 } 822 }
829 823
830 } // namespace cast 824 } // namespace cast
831 } // namespace media 825 } // namespace media
OLDNEW
« no previous file with comments | « media/cast/rtcp/rtcp_receiver_unittest.cc ('k') | media/cast/rtcp/rtcp_sender_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698