Replace deprecated base::NonThreadSafe in base in favor of SequenceChecker.

base/files/important_file_writer.cc

Index: base/files/important_file_writer.cc
diff --git a/base/files/important_file_writer.cc b/base/files/important_file_writer.cc
index 5088bd9967ba1e94420c99e29a9c44b654399a43..d7b1dce770f57ea183827daa6d8d907f3af1b685 100644
--- a/base/files/important_file_writer.cc
+++ b/base/files/important_file_writer.cc
@@ -152,11 +152,12 @@ ImportantFileWriter::ImportantFileWriter(
       serializer_(nullptr),
       commit_interval_(interval),
       weak_factory_(this) {
-  DCHECK(CalledOnValidThread());
+  DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);

[Nico, 2017/05/30 03:03:47]

Doing this in the ctor is pointless, isn't it?

[gab, 2017/05/30 14:46:58]

Indeed but the script doesn't know that :), fixed.

   DCHECK(task_runner_);
 }
 
 ImportantFileWriter::~ImportantFileWriter() {
+  DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
   // We're usually a member variable of some other object, which also tends
   // to be our serializer. It may not be safe to call back to the parent object
   // being destructed.
@@ -164,12 +165,12 @@ ImportantFileWriter::~ImportantFileWriter() {
 }
 
 bool ImportantFileWriter::HasPendingWrite() const {
-  DCHECK(CalledOnValidThread());
+  DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
   return timer().IsRunning();
 }
 
 void ImportantFileWriter::WriteNow(std::unique_ptr<std::string> data) {
-  DCHECK(CalledOnValidThread());
+  DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
   if (!IsValueInRangeForNumericType<int32_t>(data->length())) {
     NOTREACHED();
     return;
@@ -192,7 +193,7 @@ void ImportantFileWriter::WriteNow(std::unique_ptr<std::string> data) {
 }
 
 void ImportantFileWriter::ScheduleWrite(DataSerializer* serializer) {
-  DCHECK(CalledOnValidThread());
+  DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
 
   DCHECK(serializer);
   serializer_ = serializer;