Propagate audible state from player to the containing tab

content/browser/media/audio_stream_monitor_unittest.cc

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 0d206216db7c6efd763c96f0adc77fad049b1dad..5b6dab42646f6e17e70856f28a364fc0e729897a 100644
--- a/content/browser/media/audio_stream_monitor_unittest.cc
+++ b/content/browser/media/audio_stream_monitor_unittest.cc
@@ -52,7 +52,13 @@ class AudioStreamMonitorTest : public RenderViewHostTestHarness {
     WebContentsImpl* web_contents = reinterpret_cast<WebContentsImpl*>(
         RenderViewHostTestHarness::web_contents());
     web_contents->SetDelegate(&mock_web_contents_delegate_);
-    monitor_ = web_contents->audio_stream_monitor();
+
+    AudioStateProvider* provider = web_contents->audio_state_provider();
+    ASSERT_TRUE(provider);
+
+    monitor_ = provider->audio_stream_monitor();
+    ASSERT_TRUE(monitor_);

[no sievers, 2015/02/25 21:22:04]

Doesn't this fail on Android?

If we don't feel like this class will ever be useful, then you can probably just
ifdef-out the whole file on Android.

[Tima Vaisburd, 2015/02/28 03:16:06]

Yes, I did not realize this test even runs on Android. We can exclude it, but I
tried to make it work since it worked before. Please take a look at the hoops.

+
     const_cast<base::TickClock*&>(monitor_->clock_) = &clock_;
   }