cc: Task should complete as finished or canceled in tests.

cc/raster/task.cc

 // Copyright 2016 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 "cc/raster/task.h"
 
 #include "base/logging.h"
 
 namespace cc {
 
 TaskState::TaskState() : value_(Value::NEW) {}
 
 TaskState::~TaskState() {
   DCHECK(value_ != Value::RUNNING)
       << "Running task should never get destroyed.";
-  // TODO(prashant.n): Remove NEW, once all the tests follow the task life
-  // cycle correctly. Few tests still do not take care of task states.
-  // crbug.com/613814.
-  DCHECK(value_ == Value::NEW || value_ == Value::FINISHED ||
-         value_ == Value::CANCELED)
+  DCHECK(value_ == Value::FINISHED || value_ == Value::CANCELED)
       << "Task, if scheduled, should get concluded either in FINISHED or "
          "CANCELED state.";
 }
 
 bool TaskState::IsScheduled() const {
   return value_ == Value::SCHEDULED;
 }
 
 bool TaskState::IsRunning() const {
   return value_ == Value::RUNNING;
@@ skipping 50 matching lines @@
   nodes.swap(other->nodes);
   edges.swap(other->edges);
 }
 
 void TaskGraph::Reset() {
   nodes.clear();
   edges.clear();
 }
 
 }  // namespace cc