H.264 video codec support using OpenH264/FFmpeg

webrtc/modules/video_coding/codecs/h264/include/h264.h

Index: webrtc/modules/video_coding/codecs/h264/include/h264.h
diff --git a/webrtc/modules/video_coding/codecs/h264/include/h264.h b/webrtc/modules/video_coding/codecs/h264/include/h264.h
index 3f52839a6cbce59470c2cf07c5cf241c2c607608..f48cf3d09d7cc685102e47185f66b085f08cc12e 100644
--- a/webrtc/modules/video_coding/codecs/h264/include/h264.h
+++ b/webrtc/modules/video_coding/codecs/h264/include/h264.h
@@ -30,7 +30,10 @@ namespace webrtc {
 class H264Encoder : public VideoEncoder {
  public:
   static H264Encoder* Create();
+  // If H.264 is supported (any implementation).
   static bool IsSupported();
+  // If H.264 is supported with OpenH264/H264EncoderImpl implementation.
+  static bool IsSupportedOpenH264();

[hta - Chromium, 2015/10/01 09:45:51]

I can't see anyone who calls this function.
1) Is it needed?
2) If it's needed, it should be called by an unit test. Where?

[hbos, 2015/10/01 12:19:45]

My bad, it was used in a prior PS. It's not needed anymore - removing.

[stefan-webrtc, 2015/10/01 13:13:52]

Then I guess you want to delete it? :)

[hbos, 2015/11/27 14:43:42]

Oh, alright then! :)

 
   ~H264Encoder() override {}
 };
@@ -38,7 +41,10 @@ class H264Encoder : public VideoEncoder {
 class H264Decoder : public VideoDecoder {
  public:
   static H264Decoder* Create();
+  // If H.264 is supported (any implementation).
   static bool IsSupported();
+  // If H.264 is supported with OpenH264/H264DecoderImpl implementation.
+  static bool IsSupportedOpenH264();
 
   ~H264Decoder() override {}
 };