Loosening the coupling between WebRTC and //third_party/protobuf

webrtc/modules/audio_coding/audio_network_adaptor/controller_manager.h

Index: webrtc/modules/audio_coding/audio_network_adaptor/controller_manager.h
diff --git a/webrtc/modules/audio_coding/audio_network_adaptor/controller_manager.h b/webrtc/modules/audio_coding/audio_network_adaptor/controller_manager.h
index 338a3657e28ec75ba5748f7047e3f2b0332ab834..9a2c4590aeb7f1cc65b7b88f8d409b298f01a43b 100644
--- a/webrtc/modules/audio_coding/audio_network_adaptor/controller_manager.h
+++ b/webrtc/modules/audio_coding/audio_network_adaptor/controller_manager.h
@@ -16,6 +16,7 @@
 #include <vector>
 
 #include "webrtc/base/constructormagic.h"
+#include "webrtc/base/protobuf_utils.h"
 #include "webrtc/modules/audio_coding/audio_network_adaptor/controller.h"
 
 namespace webrtc {
@@ -49,7 +50,7 @@ class ControllerManagerImpl final : public ControllerManager {
   };
 
   static std::unique_ptr<ControllerManager> Create(
-      const std::string& config_string,
+      const ProtoString& config_string,

[michaelt, 2017/03/16 13:39:15]

As in other places, this ProtoString is passed to this function and was before a
std::string. We may could declare the string which is passed to here ProtoString
as well and this all the way up.

[mbonadei, 2017/03/16 14:48:57]

Done.

       size_t num_encoder_channels,
       rtc::ArrayView<const int> encoder_frame_lengths_ms,
       int min_encoder_bitrate_bps,