commit 2c72fe8a2a0e540a66028254b26e2409f9360cd8
author Ilya Nikolaevskiy <ilnik@webrtc.org> Mon Oct 02 13:01:04 2017 +0200
committer Commit Bot <commit-bot@chromium.org> Mon Oct 02 13:23:49 2017 +0000
tree 74cd91c24c13f382562bf2ae6bfc0ec5b31fcbbf
parent 8e56076bb4adf76d1fe3f789c89bab510b6aa785

Fix crash with rtc_event_log in video_loopback

Because RtcEventLog is created and stopped in different threads,
SequencedTaskChecker causes failure at the end of a test.

Bug: none
Change-Id: Ibaec3162eedebd180b101ec46a171efee5fe667e
Reviewed-on: https://webrtc-review.googlesource.com/5401
Reviewed-by: Erik Språng <sprang@webrtc.org>
Commit-Queue: Ilya Nikolaevskiy <ilnik@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#20078}

video/video_quality_test.cc

diff --git a/video/video_quality_test.cc b/video/video_quality_test.cc
index 2bfe0e3..2526ac2 100644
--- a/video/video_quality_test.cc
+++ b/video/video_quality_test.cc
@@ -1912,6 +1912,8 @@
 
   analyzer->Wait();
 
+  event_log_->StopLogging();
+
   task_queue_.SendTask([&]() {
     for (std::unique_ptr<test::VideoCapturer>& video_caputurer :
          thumbnail_capturers_)
@@ -1929,7 +1931,6 @@
     DestroyStreams();
     DestroyThumbnailStreams();
 
-    event_log_->StopLogging();
     if (graph_data_output_file)
       fclose(graph_data_output_file);