base: Change WeakPtr to use SequenceChecker instead of ThreadChecker.

base/sequence_checker_impl.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 "base/sequence_checker_impl.h"
 
-#include "base/sequenced_task_runner.h"
+#include "base/logging.h"
 
 namespace base {
 
-SequenceCheckerImpl::SequenceCheckerImpl(
-    const scoped_refptr<SequencedTaskRunner>& sequenced_task_runner)
-    : sequenced_task_runner_(sequenced_task_runner) {}
+SequenceCheckerImpl::SequenceCheckerImpl()
+    : sequence_token_assigned_(false),
+      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 (!pool_)

[darin (slow to review), 2013/07/02 13:54:23]

this probably deserves a comment to draw attention to behaving like
ThreadChecker.  i realize you already document this in the header, so just a
brief note here might be enough.

[tommycli, 2013/07/02 17:22:58]

Done.

+    return thread_checker_.CalledOnValidThread();
+
+  if (SequencedWorkerPool::Owner() != pool_)
+    return false;
+
+  return 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;
+  pool_ = NULL;
+  sequence_token_ = SequencedWorkerPool::SequenceToken();
+}
+
+void SequenceCheckerImpl::EnsureSequenceTokenAssigned() const {
+  AutoLock auto_lock(lock_);
+  if (sequence_token_assigned_)
+    return;
+
+  sequence_token_assigned_ = true;
+  pool_ = SequencedWorkerPool::Owner();
+
+  // If can't get sequence token, lock to the thread ID.
+  if (!pool_)
+    return;
+
+  bool got_token = pool_->GetCurrentThreadSequenceToken(&sequence_token_);
+  DCHECK(got_token);
 }
 
 }  // namespace base