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

Side by Side Diff: media/cast/test/sender.cc

Issue 193303002: WeakPtr destruction order cleanup: media edition. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Test application that simulates a cast sender - Data can be either generated 5 // Test application that simulates a cast sender - Data can be either generated
6 // or read from a file. 6 // or read from a file.
7 7
8 #include "base/at_exit.h" 8 #include "base/at_exit.h"
9 #include "base/file_util.h" 9 #include "base/file_util.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 292 matching lines...) Expand 10 before | Expand all | Expand 10 after
303 const VideoSenderConfig video_config_; 303 const VideoSenderConfig video_config_;
304 int audio_diff_; 304 int audio_diff_;
305 const scoped_refptr<AudioFrameInput> audio_frame_input_; 305 const scoped_refptr<AudioFrameInput> audio_frame_input_;
306 const scoped_refptr<VideoFrameInput> video_frame_input_; 306 const scoped_refptr<VideoFrameInput> video_frame_input_;
307 FILE* video_file_; 307 FILE* video_file_;
308 uint8 synthetic_count_; 308 uint8 synthetic_count_;
309 base::TickClock* const clock_; // Not owned by this class. 309 base::TickClock* const clock_; // Not owned by this class.
310 base::TimeTicks start_time_; 310 base::TimeTicks start_time_;
311 base::TimeTicks send_time_; 311 base::TimeTicks send_time_;
312 scoped_ptr<TestAudioBusFactory> audio_bus_factory_; 312 scoped_ptr<TestAudioBusFactory> audio_bus_factory_;
313
314 // NOTE: This must always be last so it's invalidated before other member
315 // variables might be accessed.
313 base::WeakPtrFactory<SendProcess> weak_factory_; 316 base::WeakPtrFactory<SendProcess> weak_factory_;
317
318 DISALLOW_COPY_AND_ASSIGN(SendProcess);
Ami GONE FROM CHROMIUM 2014/03/10 22:02:08 Call out in CL description.
DaleCurtis 2014/03/10 23:47:46 Done.
314 }; 319 };
315 320
316 } // namespace cast 321 } // namespace cast
317 } // namespace media 322 } // namespace media
318 323
319 namespace { 324 namespace {
320 void UpdateCastTransportStatus( 325 void UpdateCastTransportStatus(
321 media::cast::transport::CastTransportStatus status) {} 326 media::cast::transport::CastTransportStatus status) {}
322 327
323 void LogRawEvents( 328 void LogRawEvents(
(...skipping 203 matching lines...) Expand 10 before | Expand all | Expand 10 after
527 532
528 test_thread.message_loop_proxy()->PostTask( 533 test_thread.message_loop_proxy()->PostTask(
529 FROM_HERE, 534 FROM_HERE,
530 base::Bind(&media::cast::SendProcess::SendFrame, 535 base::Bind(&media::cast::SendProcess::SendFrame,
531 base::Unretained(send_process.get()))); 536 base::Unretained(send_process.get())));
532 537
533 io_message_loop.Run(); 538 io_message_loop.Run();
534 539
535 return 0; 540 return 0;
536 } 541 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698