media: Remove bits_per_channel from AudioDecoderConfig
This CL removes bits_per_channel in favor of just using
bytes_per_channel throughout the codebase. Misc cleanup as seen on TODO.
Bug: <none>
Change-Id: Ic8a9bc0a5c147a1bdaf46085749e4e37e99f5269
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4919653
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Commit-Queue: Vikram Pasupathy <vpasupathy@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1207187}
diff --git a/media/filters/decrypting_audio_decoder_unittest.cc b/media/filters/decrypting_audio_decoder_unittest.cc
index 926d869..335a7ae 100644
--- a/media/filters/decrypting_audio_decoder_unittest.cc
+++ b/media/filters/decrypting_audio_decoder_unittest.cc
@@ -394,12 +394,12 @@
.Times(AtMost(1))
.WillOnce(RunOnceCallback<1>(true));
- // The new config is different from the initial config in bits-per-channel,
+ // The new config is different from the initial config in bytes-per-channel,
// channel layout and samples_per_second.
AudioDecoderConfig new_config(AudioCodec::kVorbis, kSampleFormatPlanarS16,
CHANNEL_LAYOUT_5_1, 88200, EmptyExtraData(),
EncryptionScheme::kCenc);
- EXPECT_NE(new_config.bits_per_channel(), config_.bits_per_channel());
+ EXPECT_NE(new_config.bytes_per_channel(), config_.bytes_per_channel());
EXPECT_NE(new_config.channel_layout(), config_.channel_layout());
EXPECT_NE(new_config.samples_per_second(), config_.samples_per_second());
ASSERT_TRUE(new_config.is_encrypted());
@@ -416,12 +416,12 @@
.Times(AtMost(1))
.WillOnce(RunOnceCallback<1>(true));
- // The new config is different from the initial config in bits-per-channel,
+ // The new config is different from the initial config in bytes-per-channel,
// channel layout and samples_per_second.
AudioDecoderConfig new_config(AudioCodec::kVorbis, kSampleFormatPlanarS16,
CHANNEL_LAYOUT_5_1, 88200, EmptyExtraData(),
EncryptionScheme::kUnencrypted);
- EXPECT_NE(new_config.bits_per_channel(), config_.bits_per_channel());
+ EXPECT_NE(new_config.bytes_per_channel(), config_.bytes_per_channel());
EXPECT_NE(new_config.channel_layout(), config_.channel_layout());
EXPECT_NE(new_config.samples_per_second(), config_.samples_per_second());
ASSERT_FALSE(new_config.is_encrypted());