Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(81)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: net/base/backoff_entry.cc

Issue 2910473005: Deprecate NonThreadSafe in net/ in favor of SequenceChecker/ThreadChecker. (Closed)
Patch Set: fix compile Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/base/backoff_entry.h ('k') | net/base/network_change_notifier_win.h » ('j') | net/url_request/url_request_context.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/base/backoff_entry.cc
diff --git a/net/base/backoff_entry.cc b/net/base/backoff_entry.cc
index a77c5251c9e647bba85748d36a4cb4fe9664c9c6..bcf73bff7dab03fcc2d1d7e2ca16317487e3322d 100644
--- a/net/base/backoff_entry.cc
+++ b/net/base/backoff_entry.cc
@@ -28,7 +28,7 @@ BackoffEntry::BackoffEntry(const BackoffEntry::Policy* policy,
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
gab 2017/05/31 20:46:36 Done.
}
void BackoffEntry::InformOfRequest(bool succeeded) {
« no previous file with comments | « net/base/backoff_entry.h ('k') | net/base/network_change_notifier_win.h » ('j') | net/url_request/url_request_context.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698