Deprecate NonThreadSafe in components/sync in favor of SequenceChecker.

components/sync/base/weak_handle.h

Index: components/sync/base/weak_handle.h
diff --git a/components/sync/base/weak_handle.h b/components/sync/base/weak_handle.h
index 5db9810c7b4fa5b4edcdf80c3fa163bfd2cf21a7..f496918e6c7f88050c355078f37edc12a99a2908 100644
--- a/components/sync/base/weak_handle.h
+++ b/components/sync/base/weak_handle.h
@@ -18,7 +18,8 @@
 #include "base/memory/weak_ptr.h"
 #include "base/single_thread_task_runner.h"
 
-// Weak handles provides a way to refer to weak pointers from another
+// SEQUENCE_CHECKER(sequence_checker_);

[stanisc, 2017/05/30 17:00:08]

It looks like the script didn't handle this correctly. This comment makes no
sense. Also the next line has an extra //

[gab, 2017/05/31 17:42:23]

Whoops..! Spent a lot of time to make the script handle that... and somehow
messed it up in last version of script which tries to insert above comments to
insertion point (which of course doesn't work for code inside a comment!).

Thankfully this appears to be the only comment with NonThreadSafe in it :) --
guess I should have checked that first before trying hard to handle a use case
for which there was one instance and then failing to do so..!

Fixed.

+//// Weak handles provides a way to refer to weak pointers from another
 // thread.  This is useful because it is not safe to reference a weak
 // pointer from a thread other than the thread on which it was
 // created.
@@ -47,7 +48,7 @@
 //   const WeakHandle<Foo> foo_;
 // };
 //
-// class Foo : public SupportsWeakPtr<Foo>, public NonThreadSafe {
+// class Foo : public SupportsWeakPtr<Foo> {
 //  public:
 //   Foo() {
 //     SpawnFooIOWorkerOnIOThread(base::MakeWeakHandle(AsWeakPtr()));