[SUSPENDED] Scheduler: Add deprecation comments on ExecutionContext::postTask()

third_party/WebKit/Source/core/dom/Document.h

Index: third_party/WebKit/Source/core/dom/Document.h
diff --git a/third_party/WebKit/Source/core/dom/Document.h b/third_party/WebKit/Source/core/dom/Document.h
index d31b73c2933e9eba7c572af82fd298f97f0d90e6..65e67212f2ff7c47333b34437f72532274538e25 100644
--- a/third_party/WebKit/Source/core/dom/Document.h
+++ b/third_party/WebKit/Source/core/dom/Document.h
@@ -990,12 +990,13 @@ class CORE_EXPORT Document : public ContainerNode,
   bool isDNSPrefetchEnabled() const { return m_isDNSPrefetchEnabled; }
   void parseDNSPrefetchControlHeader(const String&);
 
-  // FIXME(crbug.com/305497): This should be removed once LocalDOMWindow is an
-  // ExecutionContext.
-  void postTask(const WebTraceLocation&,
-                std::unique_ptr<ExecutionContextTask>,
-                const String& taskNameForInstrumentation = emptyString())
-      override;  // Executes the task on context's thread asynchronously.
+  // DEPRECATED: Please use TaskRunnerHelper or ParentFrameTaskRunners.
+  // (https://crbug.com/624696)
+  // Executes the task on context's thread asynchronously.

[tzik, 2016/11/28 07:27:53]

There are several missing feature that TaskRunnerHelper doesn't have:
suspension, cancellation and the inspector instrumentation.
So, IMO, it's too early to deprecated this.

[nhiroki, 2016/11/28 08:07:09]

Thank you for letting me know it. Then, I'll wait until these features are
landed. Are there any crbugs to track them?

+  void postTask(
+      const WebTraceLocation&,
+      std::unique_ptr<ExecutionContextTask>,
+      const String& taskNameForInstrumentation = emptyString()) override;
   void postInspectorTask(const WebTraceLocation&,
                          std::unique_ptr<ExecutionContextTask>);