Fixes for memory and threading issues in cast

media/cast/test/sender.cc

 // Copyright 2013 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.
 
 // Test application that simulates a cast sender - Data can be either generated
 // or read from a file.
 
 #include "base/at_exit.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 290 matching lines @@
 int main(int argc, char** argv) {
   base::AtExitManager at_exit;
   VLOG(1) << "Cast Sender";
   base::Thread test_thread("Cast sender test app thread");
   base::Thread audio_thread("Cast audio encoder thread");
   base::Thread video_thread("Cast video encoder thread");
   test_thread.Start();
   audio_thread.Start();
   video_thread.Start();
 
-  base::DefaultTickClock clock;
+  scoped_ptr<base::TickClock> clock(new base::DefaultTickClock());
   base::MessageLoopForIO io_message_loop;
 
   int remote_port, local_port;
   media::cast::GetPorts(&remote_port, &local_port);
 
   std::string remote_ip_address =
       media::cast::GetIpAddress("Enter receiver IP.");
   std::string local_ip_address = media::cast::GetIpAddress("Enter local IP.");
 
   media::cast::AudioSenderConfig audio_config =
       media::cast::GetAudioSenderConfig();
   media::cast::VideoSenderConfig video_config =
       media::cast::GetVideoSenderConfig();
 
   // Setting up transport config.
   media::cast::transport::CastTransportConfig config;
   config.receiver_ip_address = remote_ip_address;
   config.local_ip_address = local_ip_address;
   config.receive_port = local_port;
   config.send_port = remote_port;
   config.audio_ssrc = audio_config.sender_ssrc;
   config.video_ssrc = video_config.sender_ssrc;
   config.audio_rtp_payload_type = audio_config.rtp_payload_type;
   config.video_rtp_payload_type = video_config.rtp_payload_type;
 
   scoped_ptr<media::cast::transport::CastTransportSender> transport_sender(
       media::cast::transport::
-      CastTransportSender::CreateCastTransportSender(&clock, config,
+      CastTransportSender::CreateCastTransportSender(clock.get(), config,
       base::Bind(&UpdateCastTransportStatus),
       io_message_loop.message_loop_proxy()));
 
   // Enable main and send side threads only. Disable logging.
   // Running transport on the main thread.
   scoped_refptr<media::cast::CastEnvironment> cast_environment(new
       media::cast::CastEnvironment(
-      &clock,
+      clock.Pass(),
       io_message_loop.message_loop_proxy(),
       audio_thread.message_loop_proxy(),
       NULL,
       video_thread.message_loop_proxy(),
       NULL,
       io_message_loop.message_loop_proxy(),
       media::cast::GetDefaultCastSenderLoggingConfig()));
 
   scoped_ptr<media::cast::CastSender> cast_sender(
       media::cast::CastSender::CreateCastSender(cast_environment,
       audio_config,
       video_config,
       NULL,  // gpu_factories.
       transport_sender.get()));
 
   transport_sender->SetPacketReceiver(cast_sender->packet_receiver());
 
   media::cast::FrameInput* frame_input = cast_sender->frame_input();
   scoped_ptr<media::cast::SendProcess> send_process(new
       media::cast::SendProcess(test_thread.message_loop_proxy(),
                                cast_environment->Clock(),
                                video_config,
                                frame_input));
 
   send_process->SendFrame();
   io_message_loop.Run();
   return 0;
 }