Introduce struct MediaConfig, with construction-time settings.

talk/app/webrtc/fakemediacontroller.h

Index: talk/app/webrtc/fakemediacontroller.h
diff --git a/talk/app/webrtc/fakemediacontroller.h b/talk/app/webrtc/fakemediacontroller.h
index 5bf3e5fcf8556b914436c948d887468241ae2fae..c2567e968ca5f742f6510b07f245125b3c03c308 100644
--- a/talk/app/webrtc/fakemediacontroller.h
+++ b/talk/app/webrtc/fakemediacontroller.h
@@ -30,6 +30,7 @@
 
 #include "talk/app/webrtc/mediacontroller.h"
 #include "webrtc/base/checks.h"
+#include "webrtc/media/base/mediachannel.h"
 
 namespace cricket {
 
@@ -46,6 +47,10 @@ class FakeMediaController : public webrtc::MediaControllerInterface {
   cricket::ChannelManager* channel_manager() const override {
     return channel_manager_;
   }
+  const MediaConfig& config() const override {
+    static const MediaConfig media_config = MediaConfig();

[perkj_webrtc, 2016/02/09 15:35:37]

nit: add as a member instead.

[nisse-webrtc, 2016/02/10 08:53:00]

I can do that, but in my opinion, a local definition is more readable.

Update: I tried it, and C++ doesn't like me... I want this object to be
statically allocated read-only data, set to the default values specified in the
declaration of the MediaConfig struct. So I wrote 

 private:
  static const MediaConfig media_config_ = MediaConfig();

I write the initializer "= MediaConfig()" because I want it properly initialized
with default values (do I get that if I drop the initializer?). And then the
compiler complains, "in-class initializer for static data member of type 'const
cricket::MediaConfig' requires 'constexpr' specifier". But I don't want any
fancy compile-time computation, I just want an initializer equivalent to writing
"= { false, true, false }" in C.

+    return media_config;
+  }
 
  private:
   cricket::ChannelManager* channel_manager_;