Adding audio network adaptor to AudioEncoderOpus.

webrtc/modules/audio_coding/audio_network_adaptor/mock/mock_controller_manager.h

Index: webrtc/modules/audio_coding/audio_network_adaptor/mock/mock_controller_manager.h
diff --git a/webrtc/modules/audio_coding/audio_network_adaptor/mock/mock_controller_manager.h b/webrtc/modules/audio_coding/audio_network_adaptor/mock/mock_controller_manager.h
index a1cc6250de6d66ea55ebfc61590db12ff83abe4d..c51967b12a62048e4ccd667bfdc67d41f36d9d7d 100644
--- a/webrtc/modules/audio_coding/audio_network_adaptor/mock/mock_controller_manager.h
+++ b/webrtc/modules/audio_coding/audio_network_adaptor/mock/mock_controller_manager.h
@@ -11,9 +11,6 @@
 #ifndef WEBRTC_MODULES_AUDIO_CODING_AUDIO_NETWORK_ADAPTOR_MOCK_MOCK_CONTROLLER_MANAGER_H_
 #define WEBRTC_MODULES_AUDIO_CODING_AUDIO_NETWORK_ADAPTOR_MOCK_MOCK_CONTROLLER_MANAGER_H_
 
-#include <memory>
-#include <vector>
-

[kwiberg-webrtc, 2016/09/27 09:35:54]

This seems unrelated to the rest of the CL. Also, why are you removing the
<vector> include when std::vector is still used below?

[minyue-webrtc, 2016/09/27 16:02:32]

They are truly unrelated. just a small cleaning-up.

vector is (and has to be) defined in controller_manager.h. Do we normally skip
include in such cases.

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

The policy is to include all the headers that are supposed to be exporting the
stuff that you're using. If you use stuff from both a.h and b.h, and b.h happens
to include a.h for some reason, it's technically possible for you to include
just b.h, but you're *supposed* to include a.h too.

There's one exception: In foo.cc, you always include foo.h first, and if both
foo.cc and foo.h use stuff from a.h, you are allowed to only include a.h in
foo.h.

 #include "webrtc/modules/audio_coding/audio_network_adaptor/controller_manager.h"
 #include "testing/gmock/include/gmock/gmock.h"