Drop irrelevant DCHECK and re-enable SyncFileSystemApiTest.GetFileStatus

storage/browser/fileapi/timed_task_helper.cc

 // 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 "storage/browser/fileapi/timed_task_helper.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/sequenced_task_runner.h"
 
 namespace storage {
 
 struct TimedTaskHelper::Tracker {
   explicit Tracker(TimedTaskHelper* timer) : timer(timer) {}
 
   ~Tracker() {
     if (timer)
       timer->tracker_ = NULL;
   }
 
   TimedTaskHelper* timer;
 };
 
 TimedTaskHelper::TimedTaskHelper(base::SequencedTaskRunner* task_runner)
     : task_runner_(task_runner),
       tracker_(NULL) {
 }
 
 TimedTaskHelper::~TimedTaskHelper() {
-  DCHECK(task_runner_->RunsTasksOnCurrentThread());

[nhiroki, 2014/10/08 05:29:55]

Let me confirm my understanding... when task_runner_ was already shutdown, this
dtor is called on another task_runner and there is no remaining task. So we can
safely destruct the helper without this check.

   if (tracker_)
     tracker_->timer = NULL;
 }
 
 bool TimedTaskHelper::IsRunning() const {
   DCHECK(task_runner_->RunsTasksOnCurrentThread());
   return tracker_ != NULL;
 }
 
 void TimedTaskHelper::Start(
@@ skipping 42 matching lines @@
 
 void TimedTaskHelper::PostDelayedTask(scoped_ptr<Tracker> tracker,
                                       base::TimeDelta delay) {
   task_runner_->PostDelayedTask(
       posted_from_,
       base::Bind(&TimedTaskHelper::Fired, base::Passed(&tracker)),
       delay);
 }
 
 }  // namespace storage