Add unit tests for avfoundation format mapper functions and fix wrong implementation.

webrtc/sdk/objc/Framework/Classes/avfoundationformatmapper.h

Index: webrtc/sdk/objc/Framework/Classes/avfoundationformatmapper.h
diff --git a/webrtc/sdk/objc/Framework/Classes/avfoundationformatmapper.h b/webrtc/sdk/objc/Framework/Classes/avfoundationformatmapper.h
index 9af55e9b0efef21d5c53bf46cd508b83838abe95..b4c69ad6c73b1f6d907b96b2c916db44dc23ab87 100644
--- a/webrtc/sdk/objc/Framework/Classes/avfoundationformatmapper.h
+++ b/webrtc/sdk/objc/Framework/Classes/avfoundationformatmapper.h
@@ -10,18 +10,18 @@
 
 #import <AVFoundation/AVFoundation.h>
 #import <Foundation/Foundation.h>
-
+#include <vector>

[magjed_webrtc, 2016/11/29 16:31:39]

nit: You should include set instead. Also, place standard library includes
before any other include, like this:
#include <set>

#import <AVFoundation/AVFoundation.h>
...

 #include "webrtc/media/base/videocapturer.h"
 
 namespace webrtc {
 // Mapping from AVCaptureDeviceFormat to cricket::VideoFormat for given input
 // device.
-extern std::set<cricket::VideoFormat> GetSupportedVideoFormatsForDevice(
+std::set<cricket::VideoFormat> GetSupportedVideoFormatsForDevice(
     AVCaptureDevice* device);
 
 // Sets device format for the provided capture device. Returns YES/NO depending
 // on success.
-extern bool SetFormatForCaptureDevice(AVCaptureDevice* device,
-                                      AVCaptureSession* session,
-                                      const cricket::VideoFormat& format);
+bool SetFormatForCaptureDevice(AVCaptureDevice* device,
+                               AVCaptureSession* session,
+                               const cricket::VideoFormat& format);
 }