using
find test -name "*.h" -o -name "*.cc" | xargs tools_webrtc/iwyu/apply_include_cleaner.py
followed by
git cl format
and
tools_webrtc/gn_check_autofix.py -C out/Default
followed by running clang-tidy with
tools/clang/scripts/build_clang_tools_extra.py \
--fetch out/Default clang-tidy clang-apply-replacements
ninja -C out/Default
gn gen out/Default --export-compile-commands
cd out/Default
tools/clang/third_party/llvm/clang-tools-extra/clang-tidy/tool/run-clang-tidy.py -p . \
-clang-tidy-binary out/Default/tools/clang/third_party/llvm/build/bin/clang-tidy \
-clang-apply-replacements-binary \
out/Default/tools/clang/third_party/llvm/build/bin/clang-apply-replacements \
-checks='-*,modernize-deprecated-headers' \
-fix
to move from C style standard headers to C++ style variants followed
another round of IWYU (which brought back stdio.h), format and gn_check_autofix.
This was followed by a manual pass removing C style headers from C++ headers when the corresponding C++-style header was in included in the .cc file. IWYU after thought brought some instances of stdio.h back *again*.
Bug: webrtc:42226242
Change-Id: I3ec5ee4bd3d4f81e25f5dfca3764bb85c2cad99b
No-Iwyu: false positive of jpeg include
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/396321
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
Commit-Queue: Philipp Hancke <phancke@meta.com>
Cr-Commit-Position: refs/heads/main@{#44937}
84 lines
2.7 KiB
C++
84 lines
2.7 KiB
C++
/*
|
|
* Copyright (c) 2018 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.
|
|
*/
|
|
|
|
#ifndef TEST_VIDEO_DECODER_PROXY_FACTORY_H_
|
|
#define TEST_VIDEO_DECODER_PROXY_FACTORY_H_
|
|
|
|
#include <cstdint>
|
|
#include <memory>
|
|
#include <vector>
|
|
|
|
#include "api/environment/environment.h"
|
|
#include "api/video/encoded_image.h"
|
|
#include "api/video_codecs/sdp_video_format.h"
|
|
#include "api/video_codecs/video_decoder.h"
|
|
#include "api/video_codecs/video_decoder_factory.h"
|
|
#include "rtc_base/checks.h"
|
|
|
|
namespace webrtc {
|
|
namespace test {
|
|
|
|
// A decoder factory with a single underlying VideoDecoder object, intended for
|
|
// test purposes. Each call to CreateVideoDecoder returns a proxy for the same
|
|
// decoder, typically an instance of FakeDecoder or MockEncoder.
|
|
class VideoDecoderProxyFactory final : public VideoDecoderFactory {
|
|
public:
|
|
explicit VideoDecoderProxyFactory(VideoDecoder* decoder)
|
|
: decoder_(decoder) {}
|
|
|
|
// Unused by tests.
|
|
std::vector<SdpVideoFormat> GetSupportedFormats() const override {
|
|
RTC_DCHECK_NOTREACHED();
|
|
return {};
|
|
}
|
|
|
|
std::unique_ptr<VideoDecoder> Create(const Environment& env,
|
|
const SdpVideoFormat& format) override {
|
|
return std::make_unique<DecoderProxy>(decoder_);
|
|
}
|
|
|
|
private:
|
|
// Wrapper class, since CreateVideoDecoder needs to surrender
|
|
// ownership to the object it returns.
|
|
class DecoderProxy final : public VideoDecoder {
|
|
public:
|
|
explicit DecoderProxy(VideoDecoder* decoder) : decoder_(decoder) {}
|
|
|
|
private:
|
|
int32_t Decode(const EncodedImage& input_image,
|
|
int64_t render_time_ms) override {
|
|
return decoder_->Decode(input_image, render_time_ms);
|
|
}
|
|
bool Configure(const Settings& settings) override {
|
|
return decoder_->Configure(settings);
|
|
}
|
|
int32_t RegisterDecodeCompleteCallback(
|
|
DecodedImageCallback* callback) override {
|
|
return decoder_->RegisterDecodeCompleteCallback(callback);
|
|
}
|
|
int32_t Release() override { return decoder_->Release(); }
|
|
DecoderInfo GetDecoderInfo() const override {
|
|
return decoder_->GetDecoderInfo();
|
|
}
|
|
const char* ImplementationName() const override {
|
|
return decoder_->ImplementationName();
|
|
}
|
|
|
|
VideoDecoder* const decoder_;
|
|
};
|
|
|
|
VideoDecoder* const decoder_;
|
|
};
|
|
|
|
} // namespace test
|
|
} // namespace webrtc
|
|
|
|
#endif // TEST_VIDEO_DECODER_PROXY_FACTORY_H_
|