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

Unified Diff: media/audio/audio_streams_tracker_unittest.cc

Issue 2655413004: Strip out stream counting from AudioRendererHost. (Closed)
Patch Set: Add moar DCHECKs. Created 3 years, 10 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
« no previous file with comments | « media/audio/audio_streams_tracker.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/audio/audio_streams_tracker_unittest.cc
diff --git a/media/audio/audio_streams_tracker_unittest.cc b/media/audio/audio_streams_tracker_unittest.cc
deleted file mode 100644
index f42142becabc4399574208ec75ba11f2a5abdc9c..0000000000000000000000000000000000000000
--- a/media/audio/audio_streams_tracker_unittest.cc
+++ /dev/null
@@ -1,101 +0,0 @@
-// Copyright 2015 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "media/audio/audio_streams_tracker.h"
-
-#include <stddef.h>
-
-#include "testing/gtest/include/gtest/gtest.h"
-
-namespace media {
-
-TEST(AudioStreamsTrackerTest, IncreaseAndDecreaseOnce) {
- AudioStreamsTracker audio_streams_tracker;
- const size_t kNumberOfOperations = 5;
-
- // Increase count.
- for (size_t i = 1; i <= kNumberOfOperations; ++i) {
- audio_streams_tracker.IncreaseStreamCount();
- EXPECT_EQ(i, audio_streams_tracker.current_stream_count_);
- EXPECT_EQ(i, audio_streams_tracker.max_stream_count());
- }
-
- // Reset and check max count.
- audio_streams_tracker.ResetMaxStreamCount();
- EXPECT_EQ(kNumberOfOperations, audio_streams_tracker.max_stream_count());
-
- // Decrease count.
- for (size_t i = 1; i <= kNumberOfOperations; ++i) {
- audio_streams_tracker.DecreaseStreamCount();
- EXPECT_EQ(kNumberOfOperations - i,
- audio_streams_tracker.current_stream_count_);
- EXPECT_EQ(kNumberOfOperations, audio_streams_tracker.max_stream_count());
- }
-
- // Reset and check max count.
- audio_streams_tracker.ResetMaxStreamCount();
- EXPECT_EQ(0u, audio_streams_tracker.max_stream_count());
-}
-
-TEST(AudioStreamsTrackerTest, IncreaseAndDecreaseMultiple) {
- AudioStreamsTracker audio_streams_tracker;
- const size_t kNumberOfOperations = 5;
-
- // Increase 2N (N = kNumberOfOperations).
- for (size_t i = 1; i <= 2 * kNumberOfOperations; ++i) {
- audio_streams_tracker.IncreaseStreamCount();
- EXPECT_EQ(i, audio_streams_tracker.current_stream_count_);
- EXPECT_EQ(i, audio_streams_tracker.max_stream_count());
- }
-
- // Decrease N.
- for (size_t i = 1; i <= kNumberOfOperations; ++i) {
- audio_streams_tracker.DecreaseStreamCount();
- EXPECT_EQ(2 * kNumberOfOperations - i,
- audio_streams_tracker.current_stream_count_);
- EXPECT_EQ(2 * kNumberOfOperations,
- audio_streams_tracker.max_stream_count());
- }
-
- // Reset and check max count.
- audio_streams_tracker.ResetMaxStreamCount();
- EXPECT_EQ(kNumberOfOperations, audio_streams_tracker.max_stream_count());
-
- // Increase 2N.
- for (size_t i = 1; i <= 2 * kNumberOfOperations; ++i) {
- audio_streams_tracker.IncreaseStreamCount();
- EXPECT_EQ(kNumberOfOperations + i,
- audio_streams_tracker.current_stream_count_);
- EXPECT_EQ(kNumberOfOperations + i,
- audio_streams_tracker.max_stream_count());
- }
-
- // Decrease N.
- for (size_t i = 1; i <= kNumberOfOperations; ++i) {
- audio_streams_tracker.DecreaseStreamCount();
- EXPECT_EQ(3 * kNumberOfOperations - i,
- audio_streams_tracker.current_stream_count_);
- EXPECT_EQ(3 * kNumberOfOperations,
- audio_streams_tracker.max_stream_count());
- }
-
- // Reset and check max count.
- audio_streams_tracker.ResetMaxStreamCount();
- EXPECT_EQ(2 * kNumberOfOperations, audio_streams_tracker.max_stream_count());
-
- // Decrease 2N.
- for (size_t i = 1; i <= 2 * kNumberOfOperations; ++i) {
- audio_streams_tracker.DecreaseStreamCount();
- EXPECT_EQ(2 * kNumberOfOperations - i,
- audio_streams_tracker.current_stream_count_);
- EXPECT_EQ(2 * kNumberOfOperations,
- audio_streams_tracker.max_stream_count());
- }
-
- // Reset and check max count.
- audio_streams_tracker.ResetMaxStreamCount();
- EXPECT_EQ(0u, audio_streams_tracker.max_stream_count());
-}
-
-} // namespace media
« no previous file with comments | « media/audio/audio_streams_tracker.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698