Replace deprecated base::NonThreadSafe in chrome/browser/safe_browsing in favor of SequenceChecker.

chrome/browser/safe_browsing/download_feedback.h

Index: chrome/browser/safe_browsing/download_feedback.h
diff --git a/chrome/browser/safe_browsing/download_feedback.h b/chrome/browser/safe_browsing/download_feedback.h
index cd8adf5edf43c58a505d6b7c759fb139c40acdde..bea6d47a96520d984d55104abe6a1a2351f4b810 100644
--- a/chrome/browser/safe_browsing/download_feedback.h
+++ b/chrome/browser/safe_browsing/download_feedback.h
@@ -11,7 +11,7 @@
 
 #include "base/callback_forward.h"
 #include "base/files/file_path.h"
-#include "base/threading/non_thread_safe.h"
+#include "base/sequence_checker.h"
 #include "chrome/browser/safe_browsing/two_phase_uploader.h"
 
 namespace safe_browsing {
@@ -20,7 +20,7 @@ class DownloadFeedbackFactory;
 
 // Handles the uploading of a single downloaded binary to the safebrowsing
 // download feedback service.
-class DownloadFeedback : public base::NonThreadSafe {
+class DownloadFeedback {

[Nathan Parker, 2017/05/30 17:16:47]

NonThreadSafe was removed, but no SequenceChecker was added here.

[gab, 2017/05/30 21:54:00]

Forgot to say in CL description but I found a few like this one where the impl
didn't use any NonThreadSafe methods... so I guess it was merely verifying on
destruction... unless the destructor was more crucial than anything, I'll leave
it up to the OWNERS to add missing checks.

[Nathan Parker, 2017/06/01 18:41:44]

I'm confused -- Why not add DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_)
to the dtor, like you did for the other file?  That way this retains the dtor
checking.

  public:
   // Takes ownership of the file pointed to be |file_path|, it will be deleted
   // when the DownloadFeedback is destructed.