Storing raw audio sink for default audio track.

talk/media/webrtc/webrtcvoiceengine_unittest.cc

 /*
  * libjingle
  * Copyright 2008 Google Inc.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are met:
  *
  *  1. Redistributions of source code must retain the above copyright notice,
  *     this list of conditions and the following disclaimer.
  *  2. Redistributions in binary form must reproduce the above copyright notice,
@@ skipping 54 matching lines @@
                             engine,  // base
                             engine,  // codec
                             engine,  // hw
                             engine,  // network
                             engine,  // rtp
                             engine) {  // volume
   }
 };
 }  // namespace
 
+class FakeAudioSink : public rtc::RefCountedObject<webrtc::AudioSinkInterface> {
+ public:
+  void OnData(const Data& audio) override {}
+};
+
 class WebRtcVoiceEngineTestFake : public testing::Test {
  public:
   WebRtcVoiceEngineTestFake()
       : call_(webrtc::Call::Config()),
         engine_(new FakeVoEWrapper(&voe_)),
         channel_(nullptr) {
     send_parameters_.codecs.push_back(kPcmuCodec);
     recv_parameters_.codecs.push_back(kPcmuCodec);
   }
   bool SetupEngine() {
@@ skipping 3153 matching lines @@
   cricket::WebRtcVoiceEngine engine;
   EXPECT_TRUE(engine.Init(rtc::Thread::Current()));
   rtc::scoped_ptr<webrtc::Call> call(
       webrtc::Call::Create(webrtc::Call::Config()));
   cricket::WebRtcVoiceMediaChannel channel(&engine, cricket::AudioOptions(),
                                            call.get());
   cricket::AudioRecvParameters parameters;
   parameters.codecs = engine.codecs();
   EXPECT_TRUE(channel.SetRecvParameters(parameters));
 }
+
+TEST_F(WebRtcVoiceEngineTestFake, SetRawAudioSink) {

[the sun, 2016/01/08 13:14:27]

nit: move these tests up to the other WebRtcVoiceEngineTestFake test cases.

[Taylor Brandstetter, 2016/01/08 19:46:58]

Done.

+  EXPECT_TRUE(SetupEngine());
+  rtc::scoped_refptr<FakeAudioSink> fake_sink = new FakeAudioSink();
+
+  // This should do nothing, since there's no recv stream yet.
+  channel_->SetRawAudioSink(kSsrc1, fake_sink);
+  // Ensure the ref count wasn't incremented.
+  EXPECT_TRUE(fake_sink->HasOneRef());
+
+  cricket::StreamParams stream;
+  stream.ssrcs.push_back(kSsrc1);
+  EXPECT_TRUE(channel_->AddRecvStream(stream));

[the sun, 2016/01/08 13:14:26]

  EXPECT_TRUE(channel_->AddRecvStream(
      cricket::StreamParams::CreateLegacy(kSsrc1)));

[Taylor Brandstetter, 2016/01/08 19:46:58]

Done.

+  // Now, the channel should latch on to the sink.
+  channel_->SetRawAudioSink(kSsrc1, fake_sink);
+  EXPECT_FALSE(fake_sink->HasOneRef());
+  EXPECT_EQ(fake_sink, call_.GetAudioReceiveStream(kSsrc1)->sink());

[the sun, 2016/01/08 13:14:26]

EXPECT_EQ(fake_sink, GetRecvStream(kSsrc1).sink());

and below.

[the sun, 2016/01/08 13:26:19]

Oops, that function isn't committed yet
(https://codereview.webrtc.org/1499903002/diff/80001/talk/media/webrtc/webrtcv...).
Keep as is or copy from the referenced CL... 

[Taylor Brandstetter, 2016/01/08 19:46:58]

Ok, copied.

+
+  // Setting a nullptr should release the reference.
+  channel_->SetRawAudioSink(kSsrc1, nullptr);
+  EXPECT_TRUE(fake_sink->HasOneRef());
+}
+
+TEST_F(WebRtcVoiceEngineTestFake, SetRawAudioSinkDefaultRecvStream) {
+  EXPECT_TRUE(SetupEngine());
+  rtc::scoped_refptr<FakeAudioSink> fake_sink_1 = new FakeAudioSink();
+  rtc::scoped_refptr<FakeAudioSink> fake_sink_2 = new FakeAudioSink();
+
+  // Should be able to set a default sink even when no stream exists.
+  channel_->SetRawAudioSink(0, fake_sink_1);
+  EXPECT_FALSE(fake_sink_1->HasOneRef());
+

[the sun, 2016/01/08 13:14:26]

nit: Add comment:
// Create default channel.

[Taylor Brandstetter, 2016/01/08 19:46:58]

Done.

+  DeliverPacket(kPcmuFrame, sizeof(kPcmuFrame));
+  EXPECT_EQ(fake_sink_1, call_.GetAudioReceiveStream(0x01)->sink());
+
+  // Should be able to set the default sink after a stream exists.
+  channel_->SetRawAudioSink(0, fake_sink_2);
+  EXPECT_TRUE(fake_sink_1->HasOneRef());
+  EXPECT_FALSE(fake_sink_2->HasOneRef());
+  EXPECT_EQ(fake_sink_2, call_.GetAudioReceiveStream(0x01)->sink());
+
+  // If we remove and add a default stream, it should get the same sink.
+  EXPECT_TRUE(channel_->RemoveRecvStream(0x01));
+  DeliverPacket(kPcmuFrame, sizeof(kPcmuFrame));
+  EXPECT_FALSE(fake_sink_2->HasOneRef());
+  EXPECT_EQ(fake_sink_2, call_.GetAudioReceiveStream(0x01)->sink());
+
+  // Finally, try resetting the default sink.
+  channel_->SetRawAudioSink(0, nullptr);
+  EXPECT_TRUE(fake_sink_2->HasOneRef());
+}