Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(340)

Unified Diff: media/audio/win/audio_output_win_unittest.cc

Issue 1215643003: Remove -Wno-unused-private-field clang warning suppression. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Clean up GN Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/audio/win/audio_output_win_unittest.cc
diff --git a/media/audio/win/audio_output_win_unittest.cc b/media/audio/win/audio_output_win_unittest.cc
index 3ef817b5eb5a9c63f6d94005a07d3992cf8ee1a4..c6c9cc12db09046d08b00779f6428192147af6fb 100644
--- a/media/audio/win/audio_output_win_unittest.cc
+++ b/media/audio/win/audio_output_win_unittest.cc
@@ -80,8 +80,8 @@ const int kMaxNumBuffers = 3;
// in the OnMoreData callback.
class TestSourceLaggy : public TestSourceBasic {
public:
- TestSourceLaggy(int laggy_after_buffer, int lag_in_ms)
- : laggy_after_buffer_(laggy_after_buffer), lag_in_ms_(lag_in_ms) {
+ TestSourceLaggy(int lag_in_ms)
Nico 2015/07/09 01:58:16 explicit
benwells 2015/07/09 11:43:02 Done.
+ : lag_in_ms_(lag_in_ms) {
}
int OnMoreData(AudioBus* audio_bus, uint32 total_bytes_delay) override {
// Call the base, which increments the callback_count_.
@@ -92,7 +92,6 @@ class TestSourceLaggy : public TestSourceBasic {
return audio_bus->frames();
}
private:
- int laggy_after_buffer_;
int lag_in_ms_;
};
@@ -238,7 +237,7 @@ TEST(WinAudioTest, PCMWaveSlowSource) {
16000, 16, 256),
std::string());
ASSERT_TRUE(NULL != oas);
- TestSourceLaggy test_laggy(2, 90);
+ TestSourceLaggy test_laggy(90);
EXPECT_TRUE(oas->Open());
// The test parameters cause a callback every 32 ms and the source is
// sleeping for 90 ms, so it is guaranteed that we run out of ready buffers.

Powered by Google App Engine
This is Rietveld 408576698