Deprecate NonThreadSafe in net/ in favor of SequenceChecker/ThreadChecker.

net/base/backoff_entry.cc

 // Copyright (c) 2012 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.
 
 #include "net/base/backoff_entry.h"
 
 #include <algorithm>
 #include <cmath>
 #include <limits>
 
@@ skipping 10 matching lines @@
 BackoffEntry::BackoffEntry(const BackoffEntry::Policy* policy,
                            base::TickClock* clock)
     : policy_(policy), clock_(clock) {
   DCHECK(policy_);
   Reset();
 }
 
 BackoffEntry::~BackoffEntry() {
   // TODO(joi): Remove this once our clients (e.g. URLRequestThrottlerManager)
   // always destroy from the I/O thread.
-  DetachFromThread();
+  DETACH_FROM_THREAD(thread_checker_);

[mmenke, 2017/05/31 19:28:08]

Is this needed?  thread_checker_ doesn't check on destruction, right?  Also need
to update the TODO to be about adding
DCHECK_CALLED_ON_VALID_THREAD(thread_checker_)

[gab, 2017/05/31 20:46:36]

Done.

 }
 
 void BackoffEntry::InformOfRequest(bool succeeded) {
   if (!succeeded) {
     ++failure_count_;
     exponential_backoff_release_time_ = CalculateReleaseTime();
   } else {
     // We slowly decay the number of times delayed instead of
     // resetting it to 0 in order to stay stable if we receive
     // successes interleaved between lots of failures.  Note that in
@@ skipping 135 matching lines @@
   // for overflow with both.
   int64_t release_time_us = std::min(calculated_release_time_us.ValueOrDefault(
                                          std::numeric_limits<int64_t>::max()),
                                      maximum_release_time_us.ValueOrDefault(
                                          std::numeric_limits<int64_t>::max()));
 
   return base::TimeTicks() + base::TimeDelta::FromMicroseconds(release_time_us);
 }
 
 }  // namespace net