Changed AudioPacket data to a repeated field.

remoting/proto/audio.proto

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Protocol for audio messages.
 
 syntax = "proto2";
 
 option optimize_for = LITE_RUNTIME;
 
 package remoting;
 
 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;
     ENCODING_RAW = 0;  // Uncompressed encoding
     ENCODING_OGG = 1;
   }
 
   optional Encoding encoding = 3 [default = ENCODING_INVALID];
 
   enum SamplingRate {
@@ skipping 14 matching lines @@
 
   enum Channels {
     CHANNELS_INVALID = -1;
     CHANNELS_MONO = 1;
     CHANNELS_STEREO = 2;
   }
 
   optional Channels channels = 6 [default = CHANNELS_INVALID];
 }