cc: Remove ScheduleOnOriginThread() and CompleteOnOriginThread().

cc/raster/task_graph_runner.h

 // Copyright 2014 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.
 
 #ifndef CC_RASTER_TASK_GRAPH_RUNNER_H_
 #define CC_RASTER_TASK_GRAPH_RUNNER_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <algorithm>
 #include <map>
 #include <memory>
 #include <vector>
 
 #include "base/logging.h"
 #include "base/memory/ref_counted.h"
 #include "cc/base/cc_export.h"
 
 namespace cc {
 
 class TaskGraphRunner;
 
+typedef uint32_t TaskTypeId;
+const TaskTypeId kDefaultTaskTypeId = 0;
+
 // A task which can be run by a TaskGraphRunner. To run a Task, it should be
 // inserted into a TaskGraph, which can then be scheduled on the
 // TaskGraphRunner.
 class CC_EXPORT Task : public base::RefCountedThreadSafe<Task> {
  public:
   typedef std::vector<scoped_refptr<Task>> Vector;
 
+  TaskTypeId GetTaskTypeId();

[prashant.n, 2016/04/11 14:17:48]

This looks little wierd. But I don't see any other approach. Pls. comment on
this if this is not okay?

Basically what CompleteOnOriginThread was doing is job of Task owner (viz.
TileManager, [Software|Gpu]ImageDecodeController). So we need to move those out
of Task interfaces making Task leaner.

+
   // Subclasses should implement this method. RunOnWorkerThread may be called
   // on any thread, and subclasses are responsible for locking and thread
   // safety.
   virtual void RunOnWorkerThread() = 0;
 
   void WillRun();
   void DidRun();
   bool HasFinishedRunning() const;
 
  protected:
   friend class base::RefCountedThreadSafe<Task>;
 
+  void SetTaskTypeId(TaskTypeId type_id);
+
   Task();
   virtual ~Task();
 
+  TaskTypeId type_id_;
   bool will_run_;
   bool did_run_;
 };
 
 // A task dependency graph describes the order in which to execute a set
 // of tasks. Dependencies are represented as edges. Each node is assigned
 // a category, a priority and a run count that matches the number of
 // dependencies. Priority range from 0 (most favorable scheduling) to UINT16_MAX
 // (least favorable). Categories range from 0 to UINT16_MAX. It is up to the
 // implementation and its consumer to determine the meaning (if any) of a
@@ skipping 161 matching lines @@
  private:
   TaskGraph* graph_;
   const Task* task_;
   size_t current_index_;
   TaskGraph::Node* current_node_;
 };
 
 }  // namespace cc
 
 #endif  // CC_RASTER_TASK_GRAPH_RUNNER_H_