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

Side by Side Diff: webrtc/video/rtc_event_log.proto

Issue 1340283002: Added support for logging the SSRC corresponding to AudioPlayout events. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Added missing include and changed CHECK_EQ to RTC_CHECK_EQ. Created 5 years, 3 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 | « webrtc/video/rtc_event_log.cc ('k') | webrtc/video/rtc_event_log_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 syntax = "proto2"; 1 syntax = "proto2";
2 option optimize_for = LITE_RUNTIME; 2 option optimize_for = LITE_RUNTIME;
3 package webrtc.rtclog; 3 package webrtc.rtclog;
4 4
5 5
6 enum MediaType { 6 enum MediaType {
7 ANY = 0; 7 ANY = 0;
8 AUDIO = 1; 8 AUDIO = 1;
9 VIDEO = 2; 9 VIDEO = 2;
10 DATA = 3; 10 DATA = 3;
(...skipping 88 matching lines...) Expand 10 before | Expand all | Expand 10 after
99 // AUDIO_PLAYOUT indicates a call to the PlayoutData10Ms() function in ACM. 99 // AUDIO_PLAYOUT indicates a call to the PlayoutData10Ms() function in ACM.
100 enum EventType { 100 enum EventType {
101 UNKNOWN_EVENT = 0; 101 UNKNOWN_EVENT = 0;
102 LOG_START = 1; 102 LOG_START = 1;
103 LOG_END = 2; 103 LOG_END = 2;
104 AUDIO_PLAYOUT = 3; 104 AUDIO_PLAYOUT = 3;
105 } 105 }
106 106
107 // required 107 // required
108 optional EventType type = 1; 108 optional EventType type = 1;
109
110 // required if type == AUDIO_PLAYOUT
111 optional uint32 local_ssrc = 2;
109 } 112 }
110 113
111 114
112 // TODO(terelius): Video and audio streams could in principle share SSRC, 115 // TODO(terelius): Video and audio streams could in principle share SSRC,
113 // so identifying a stream based only on SSRC might not work. 116 // so identifying a stream based only on SSRC might not work.
114 // It might be better to use a combination of SSRC and media type 117 // It might be better to use a combination of SSRC and media type
115 // or SSRC and port number, but for now we will rely on SSRC only. 118 // or SSRC and port number, but for now we will rely on SSRC only.
116 message VideoReceiveConfig { 119 message VideoReceiveConfig {
117 // required - Synchronization source (stream identifier) to be received. 120 // required - Synchronization source (stream identifier) to be received.
118 optional uint32 remote_ssrc = 1; 121 optional uint32 remote_ssrc = 1;
(...skipping 100 matching lines...) Expand 10 before | Expand all | Expand 10 after
219 222
220 223
221 message AudioReceiveConfig { 224 message AudioReceiveConfig {
222 // TODO(terelius): Add audio-receive config. 225 // TODO(terelius): Add audio-receive config.
223 } 226 }
224 227
225 228
226 message AudioSendConfig { 229 message AudioSendConfig {
227 // TODO(terelius): Add audio-receive config. 230 // TODO(terelius): Add audio-receive config.
228 } 231 }
OLDNEW
« no previous file with comments | « webrtc/video/rtc_event_log.cc ('k') | webrtc/video/rtc_event_log_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698