Commit 95bc8b67 authored by Jun Cai's avatar Jun Cai Committed by Commit Bot

Convert base::Bind and base::Callback in //media/muxers/ to Once or Repeating

This CL converts base::Bind and base::Callback in //media/muxers/ to
Once or Repeating

Bug: 1007809
Change-Id: I9d412676b0c9386de89d400e2126af2e20745821
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2065333Reviewed-by: default avatarMiguel Casas <mcasas@chromium.org>
Commit-Queue: Jun Cai <juncai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743152}
parent 92ca143e
...@@ -48,7 +48,7 @@ class MEDIA_EXPORT WebmMuxer : public mkvmuxer::IMkvWriter { ...@@ -48,7 +48,7 @@ class MEDIA_EXPORT WebmMuxer : public mkvmuxer::IMkvWriter {
public: public:
// Callback to be called when WebmMuxer is ready to write a chunk of data, // Callback to be called when WebmMuxer is ready to write a chunk of data,
// either any file header or a SingleBlock. // either any file header or a SingleBlock.
using WriteDataCB = base::Callback<void(base::StringPiece)>; using WriteDataCB = base::RepeatingCallback<void(base::StringPiece)>;
// Container for the parameters that muxer uses that is extracted from // Container for the parameters that muxer uses that is extracted from
// media::VideoFrame. // media::VideoFrame.
......
...@@ -59,7 +59,8 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size) { ...@@ -59,7 +59,8 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size) {
const auto audio_codec = static_cast<media::AudioCodec>( const auto audio_codec = static_cast<media::AudioCodec>(
kSupportedAudioCodecs[rng() % base::size(kSupportedAudioCodecs)]); kSupportedAudioCodecs[rng() % base::size(kSupportedAudioCodecs)]);
media::WebmMuxer muxer(audio_codec, input_type.has_video, media::WebmMuxer muxer(audio_codec, input_type.has_video,
input_type.has_audio, base::Bind(&OnWriteCallback)); input_type.has_audio,
base::BindRepeating(&OnWriteCallback));
base::RunLoop run_loop; base::RunLoop run_loop;
run_loop.RunUntilIdle(); run_loop.RunUntilIdle();
......
...@@ -50,7 +50,8 @@ class WebmMuxerTest : public TestWithParam<TestParams> { ...@@ -50,7 +50,8 @@ class WebmMuxerTest : public TestWithParam<TestParams> {
GetParam().audio_codec, GetParam().audio_codec,
GetParam().num_video_tracks, GetParam().num_video_tracks,
GetParam().num_audio_tracks, GetParam().num_audio_tracks,
base::Bind(&WebmMuxerTest::WriteCallback, base::Unretained(this)))), base::BindRepeating(&WebmMuxerTest::WriteCallback,
base::Unretained(this)))),
last_encoded_length_(0), last_encoded_length_(0),
accumulated_position_(0) { accumulated_position_(0) {
EXPECT_EQ(webm_muxer_->Position(), 0); EXPECT_EQ(webm_muxer_->Position(), 0);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment