Changed AudioPacket data to a repeated field.

remoting/proto/audio.proto

Index: remoting/proto/audio.proto
diff --git a/remoting/proto/audio.proto b/remoting/proto/audio.proto
index eabfcfeb55b31b949a3a070ac6161709ec921f5b..7bfc37f04b20e2e7df8a3babb7100c0e53460c1c 100644
--- a/remoting/proto/audio.proto
+++ b/remoting/proto/audio.proto
@@ -14,7 +14,9 @@ message AudioPacket {
   optional int32 timestamp = 1 [default = 0];
 
   // Data is always signed and represented using little endian.

[Sergey Ulanov, 2012/08/14 18:50:05]

this makes sense only for raw audio packets. Update the comment to make it
clear. And meaybe move it after the following sentence.

[kxing, 2012/08/14 20:56:15]

Done.

-  optional bytes data = 2;
+  // Raw data will have exactly one data field.

[Sergey Ulanov, 2012/08/14 18:50:05]

s/Raw data will/Packets with raw data must/

[kxing, 2012/08/14 20:56:15]

Done.

+  // For encoded data, each data field represents one frame.

[Sergey Ulanov, 2012/08/14 18:50:05]

It may not be necessary to frame data, depending on encoder.
Suggest rewording: Some encoders (e.g. Speex) may add multiple data fields to
separate encoded frames.

[kxing, 2012/08/14 20:56:15]

Done.

+  repeated bytes data = 2;
 
   enum Encoding {
     ENCODING_INVALID = -1;