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

Unified Diff: content/browser/media/audio_stream_monitor_unittest.cc

Issue 2698813007: Fix teardown of stale AudioStreamMonitor poll callbacks. (Closed)
Patch Set: 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
Index: content/browser/media/audio_stream_monitor_unittest.cc
diff --git a/content/browser/media/audio_stream_monitor_unittest.cc b/content/browser/media/audio_stream_monitor_unittest.cc
index f1d5103d1ce13849be5b3f307657020a445c0083..35d80540459d62c0686b398f83830e9693df7973 100644
--- a/content/browser/media/audio_stream_monitor_unittest.cc
+++ b/content/browser/media/audio_stream_monitor_unittest.cc
@@ -343,4 +343,16 @@ TEST_F(AudioStreamMonitorTest, MultipleRendererProcesses) {
ExpectIsPolling(kAnotherRenderProcessId, kStreamId, false);
}
+TEST_F(AudioStreamMonitorTest, RenderProcessGone) {
+ StartMonitoring(kRenderProcessId, kStreamId, CreatePollCallback(kStreamId));
+ StartMonitoring(kAnotherRenderProcessId, kStreamId,
+ CreatePollCallback(kStreamId));
+ ExpectIsPolling(kRenderProcessId, kStreamId, true);
+ ExpectIsPolling(kAnotherRenderProcessId, kStreamId, true);
+ monitor_->RenderProcessGone(kRenderProcessId);
+ ExpectIsPolling(kRenderProcessId, kStreamId, false);
+ monitor_->RenderProcessGone(kAnotherRenderProcessId);
+ ExpectIsPolling(kAnotherRenderProcessId, kStreamId, false);
+}
+
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698