[Chromecast] Remove usage of nonreentrant functions.

chromecast/crash/linux/minidump_writer.h

 // Copyright 2015 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.
 
 #ifndef CHROMECAST_CRASH_LINUX_MINIDUMP_WRITER_H_
 #define CHROMECAST_CRASH_LINUX_MINIDUMP_WRITER_H_
 
 #include "base/callback.h"
 #include "base/files/file_path.h"
 #include "base/macros.h"
@@ skipping 28 matching lines @@
   // |dump_state_cb_| is used inside DoWork().
   MinidumpWriter(MinidumpGenerator* minidump_generator,
                  const std::string& minidump_filename,
                  const MinidumpParams& params);
 
   ~MinidumpWriter() override;
 
   // Acquires exclusive access to the minidumps directory and generates a
   // minidump and a log to be uploaded later. Returns 0 if successful, -1
   // otherwise.
-  int Write() { return AcquireLockAndDoWork(); }
+  int Write() { return AcquireLockAndDoWork() ? 0 : -1; }
 
  protected:
   // MinidumpManager implementation:
-  int DoWork() override;
+  bool DoWork() override;
 
  private:
   MinidumpGenerator* const minidump_generator_;
   base::FilePath minidump_path_;
   const MinidumpParams params_;
 
   // This callback is Run() to dump a log to |minidump_path_|.txt.gz, taking
   // |minidump_path_| as a parameter. It returns 0 on success, and a negative
   // integer otherwise. If a callback is not passed in the constructor, the
   // default implemementaion is used.
   DumpStateCallback dump_state_cb_;
 
   DISALLOW_COPY_AND_ASSIGN(MinidumpWriter);
 };
 
 }  // namespace chromecast
 
 #endif  // CHROMECAST_CRASH_LINUX_MINIDUMP_WRITER_H_