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

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

Issue 136903003: cast: Wire upp logging to be sent over RTCP between receiver and sender. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Merge TOT Created 6 years, 11 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.cc ('k') | media/cast/rtcp/rtcp_receiver_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_receiver.h" 5 #include "media/cast/rtcp/rtcp_receiver.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "media/cast/rtcp/rtcp_utility.h" 8 #include "media/cast/rtcp/rtcp_utility.h"
9 #include "media/cast/transport/cast_transport_defines.h" 9 #include "media/cast/transport/cast_transport_defines.h"
10 10
11 namespace { 11 namespace {
12 12
13 media::cast::CastLoggingEvent TranslateToLogEventFromWireFormat(uint8 event) { 13 media::cast::CastLoggingEvent TranslateToLogEventFromWireFormat(uint8 event) {
14 switch (event) { 14 switch (event) {
15 case 1: 15 case 1:
16 return media::cast::kAckSent; 16 return media::cast::kAudioAckSent;
17 case 2: 17 case 2:
18 return media::cast::kAudioPlayoutDelay; 18 return media::cast::kAudioPlayoutDelay;
19 case 3: 19 case 3:
20 return media::cast::kAudioFrameDecoded; 20 return media::cast::kAudioFrameDecoded;
21 case 4: 21 case 4:
22 return media::cast::kAudioPacketReceived;
23 case 5:
24 return media::cast::kVideoAckSent;
25 case 6:
22 return media::cast::kVideoFrameDecoded; 26 return media::cast::kVideoFrameDecoded;
23 case 5: 27 case 7:
24 return media::cast::kVideoRenderDelay; 28 return media::cast::kVideoRenderDelay;
25 case 6: 29 case 8:
26 return media::cast::kPacketReceived; 30 return media::cast::kVideoPacketReceived;
27 case 7: 31 case 9:
28 return media::cast::kDuplicatePacketReceived; 32 return media::cast::kDuplicatePacketReceived;
29 default: 33 default:
30 // If the sender adds new log messages we will end up here until we add 34 // If the sender adds new log messages we will end up here until we add
31 // the new messages in the receiver. 35 // the new messages in the receiver.
32 VLOG(1) << "Unexpected log message received: " << static_cast<int>(event); 36 VLOG(1) << "Unexpected log message received: " << static_cast<int>(event);
33 NOTREACHED(); 37 NOTREACHED();
34 return media::cast::kUnknown; 38 return media::cast::kUnknown;
35 } 39 }
36 } 40 }
37 41
(...skipping 542 matching lines...) Expand 10 before | Expand all | Expand 10 after
580 584
581 void RtcpReceiver::HandleFIRItem(const RtcpField* rtcp_field) { 585 void RtcpReceiver::HandleFIRItem(const RtcpField* rtcp_field) {
582 // Is it our sender that is requested to generate a new keyframe. 586 // Is it our sender that is requested to generate a new keyframe.
583 if (ssrc_ != rtcp_field->fir_item.ssrc) return; 587 if (ssrc_ != rtcp_field->fir_item.ssrc) return;
584 588
585 VLOG(1) << "Cast RTCP received FIR on our SSRC " << ssrc_; 589 VLOG(1) << "Cast RTCP received FIR on our SSRC " << ssrc_;
586 } 590 }
587 591
588 } // namespace cast 592 } // namespace cast
589 } // namespace media 593 } // namespace media
OLDNEW
« no previous file with comments | « media/cast/rtcp/rtcp.cc ('k') | media/cast/rtcp/rtcp_receiver_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698