H.264 video codec support using OpenH264/FFmpeg

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

 /*
  *  Copyright (c) 2015 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  *
  */
@@ skipping 12 matching lines @@
 
 #endif  // defined(WEBRTC_IOS) || defined(WEBRTC_MAC)
 
 #include "webrtc/modules/video_coding/codecs/interface/video_codec_interface.h"
 
 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 {}
 };
 
 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 {}
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_MODULES_VIDEO_CODING_CODECS_H264_INCLUDE_H264_H_