Creating controller manager from config string in audio network adaptor.

webrtc/modules/audio_coding/audio_network_adaptor/config.proto

+syntax = "proto2";
+option optimize_for = LITE_RUNTIME;
+package webrtc.audio_network_adaptor.config;
+
+message FecController {
+  message Threshold {
+    optional int32 low_bandwidth_bps = 1;
+    optional float low_bandwidth_packet_loss = 2;
+    optional int32 high_bandwidth_bps = 3;
+    optional float high_bandwidth_packet_loss = 4;
+  }
+  optional Threshold fec_enabling_threshold = 1;
+  optional Threshold fec_disabling_threshold = 2;
+  optional int32 time_constant_ms = 3;
+}
+
+message FrameLengthController {
+  optional float fl_increasing_packet_loss_fraction = 1;
+  optional float fl_decreasing_packet_loss_fraction = 2;
+  optional int32 fl_20ms_to_60ms_bandwidth_bps = 3;
+  optional int32 fl_60ms_to_20ms_bandwidth_bps = 4;
+}
+
+message ChannelController {
+  optional int32 channel_1_to_2_bandwidth_bps = 1;
+  optional int32 channel_2_to_1_bandwidth_bps = 2;
+}
+
+message DtxController {
+  optional int32 dtx_enabling_bandwidth_bps = 1;
+  optional int32 dtx_disabling_bandwidth_bps = 2;
+}
+
+message BitrateController {}
+
+message Controller {
+  message ScoringPoint {
+    optional int32 uplink_bandwidth_bps = 1;
+    optional float uplink_packet_loss_fraction = 2;
+  }
+  optional ScoringPoint scoring_point = 1;
+  oneof controller {
+    FecController fec_controller = 21;
+    FrameLengthController frame_length_controller = 22;
+    ChannelController channel_controller = 23;
+    DtxController dtx_controller = 24;
+    BitrateController bitrate_controller = 25;
+  }
+}
+
+message ControllerManager {
+  repeated Controller controllers = 1;
+  optional int32 min_reordering_time_ms = 2;
+  optional float min_reordering_squared_distance = 3;
+}

[kwiberg-webrtc, 2016/09/28 08:59:22]

I think I see why it makes sense to have this as a protobuf instead of a struct:
complicated structure, and you'll probably want to add new knobs and stuff in
the future. Was that the reason?

Since this is part of the API, it should probably be quite heavily commented;
this is *the* place where all of these knobs should be explained.

Also, make sure that someone with protobuf experience takes a look at it (I'm
not such a person).

[minyue-webrtc, 2016/09/28 12:47:25]

Right, we also would like it to be platform and language independent.
Good point, will do.
Good point, will do.