commit 1b5e5db842971340eb9128985ddbaf0225a9d0b1
author Magnus Jedvert <magjed@webrtc.org> Thu Aug 23 10:39:22 2018
committer Commit Bot <commit-bot@chromium.org> Thu Aug 23 11:04:27 2018
tree 53c4639ca5a9858dd240b6d05e0088d72a1025df
parent 4ff63cc9a164236200992830e0f5bf71426c4464

Remove SequencedTaskChecker from Y4mFileReader

SequencedTaskChecker is not part of rtc_base_approved and will not work
in Chromium. This CL simply removes it since it was just a precaution
and is not necessary for the tool. The thread assumptions are stated in
the class comment.

TBR=phensman@webrtc.org

Bug: webrtc:9642
Change-Id: I871ac361975595d8ed07b2e2447e3581c9ba9968
Reviewed-on: https://webrtc-review.googlesource.com/95648
Reviewed-by: Magnus Jedvert <magjed@webrtc.org>
Commit-Queue: Magnus Jedvert <magjed@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24401}

rtc_tools/BUILD.gn

diff --git a/rtc_tools/BUILD.gn b/rtc_tools/BUILD.gn
index f6b03da..e707bfc 100644
--- a/rtc_tools/BUILD.gn
+++ b/rtc_tools/BUILD.gn
@@ -69,7 +69,6 @@
     "../api/video:video_frame",
     "../api/video:video_frame_i420",
     "../rtc_base:rtc_base_approved",
-    "../rtc_base:sequenced_task_checker",
     "//third_party/abseil-cpp/absl/types:optional",
   ]
 }

rtc_tools/y4m_file_reader.cc

diff --git a/rtc_tools/y4m_file_reader.cc b/rtc_tools/y4m_file_reader.cc
index e2bc399..ea8a741 100644
--- a/rtc_tools/y4m_file_reader.cc
+++ b/rtc_tools/y4m_file_reader.cc
@@ -20,7 +20,6 @@
 #include "api/video/i420_buffer.h"
 #include "rtc_base/logging.h"
 #include "rtc_base/refcountedobject.h"
-#include "rtc_base/sequenced_task_checker.h"
 #include "rtc_base/string_to_number.h"
 #include "rtc_base/stringencode.h"
 #include "rtc_base/stringutils.h"
@@ -184,7 +183,6 @@
 
 rtc::scoped_refptr<I420BufferInterface> Y4mFile::GetFrame(
     size_t frame_index) const {
-  RTC_DCHECK_CALLED_SEQUENTIALLY(&thread_checker_);
   RTC_CHECK_LT(frame_index, frame_positions_.size());
 
   fsetpos(file_, &frame_positions_[frame_index]);

rtc_tools/y4m_file_reader.h

diff --git a/rtc_tools/y4m_file_reader.h b/rtc_tools/y4m_file_reader.h
index b11beef..cb86c9e 100644
--- a/rtc_tools/y4m_file_reader.h
+++ b/rtc_tools/y4m_file_reader.h
@@ -17,7 +17,6 @@
 
 #include "api/video/video_frame.h"
 #include "rtc_base/refcount.h"
-#include "rtc_base/sequenced_task_checker.h"
 
 namespace webrtc {
 namespace test {
@@ -89,7 +88,6 @@
   const int height_;
   const float fps_;
   const std::vector<fpos_t> frame_positions_;
-  const rtc::SequencedTaskChecker thread_checker_;
   FILE* const file_;
 };