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

Unified Diff: base/sequence_checker_impl.cc

Issue 18231002: base: Change WeakPtr to use SequenceChecker instead of ThreadChecker. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 6 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
Index: base/sequence_checker_impl.cc
diff --git a/base/sequence_checker_impl.cc b/base/sequence_checker_impl.cc
index 24d9ed99a147c7c5a8d2db69557b3f891851cfa4..ff2811da8ae95330a2e215f9101d100bbe3a16e6 100644
--- a/base/sequence_checker_impl.cc
+++ b/base/sequence_checker_impl.cc
@@ -1,31 +1,52 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright 2013 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 "base/sequence_checker_impl.h"
-#include "base/sequenced_task_runner.h"
-
namespace base {
-SequenceCheckerImpl::SequenceCheckerImpl(
- const scoped_refptr<SequencedTaskRunner>& sequenced_task_runner)
- : sequenced_task_runner_(sequenced_task_runner) {}
+SequenceCheckerImpl::SequenceCheckerImpl()
+ : sequence_token_assigned_(false),
+ associated_pool_(NULL) {
+ EnsureSequenceTokenAssigned();
+}
SequenceCheckerImpl::~SequenceCheckerImpl() {}
bool SequenceCheckerImpl::CalledOnValidSequence() const {
+ EnsureSequenceTokenAssigned();
AutoLock auto_lock(lock_);
- return sequenced_task_runner_.get() ?
- sequenced_task_runner_->RunsTasksOnCurrentThread() :
- thread_checker_.CalledOnValidThread();
+
+ if (!associated_pool_)
+ return thread_checker_.CalledOnValidThread();
+
+ if (SequencedWorkerPool::Get() != associated_pool_)
+ return false;
+
+ return associated_pool_->IsRunningSequenceOnCurrentThread(sequence_token_);
}
-void SequenceCheckerImpl::ChangeSequence(
- const scoped_refptr<SequencedTaskRunner>& sequenced_task_runner) {
+void SequenceCheckerImpl::DetachFromSequence() {
AutoLock auto_lock(lock_);
- sequenced_task_runner_ = sequenced_task_runner;
thread_checker_.DetachFromThread();
+ sequence_token_assigned_ = false;
+ associated_pool_ = NULL;
+ sequence_token_ = SequencedWorkerPool::SequenceToken();
+}
+
+void SequenceCheckerImpl::EnsureSequenceTokenAssigned() const {
+ AutoLock auto_lock(lock_);
+ if (sequence_token_assigned_)
+ return;
+
+ sequence_token_assigned_ = true;
+ associated_pool_ = SequencedWorkerPool::Get();
+
+ // If can't get sequence token, lock to the thread ID.
vandebo (ex-Chrome) 2013/06/28 21:00:06 If you got a pool, probably should DCHECK that you
tommycli 2013/06/28 22:37:07 Done.
+ if (!associated_pool_ ||
+ !associated_pool_->GetCurrentThreadSequenceToken(&sequence_token_))
+ associated_pool_ = NULL;
}
} // namespace base

Powered by Google App Engine
This is Rietveld 408576698