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

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

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All
  * rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
@@ skipping 972 matching lines @@
   // Returns the HTMLLinkElement currently in use for the Web Manifest.
   // Returns null if there is no such element.
   HTMLLinkElement* linkManifest() const;
 
   void updateFocusAppearanceSoon(SelectionBehaviorOnFocus);
   void cancelFocusAppearanceUpdate();
 
   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>);
 
   void tasksWereSuspended() final;
   void tasksWereResumed() final;
   void suspendScheduledTasks() final;
   void resumeScheduledTasks() final;
   bool tasksNeedSuspension() final;
 
   void finishedParsing();
@@ skipping 708 matching lines @@
 DEFINE_TYPE_CASTS(TreeScope, Document, document, true, true);
 
 }  // namespace blink
 
 #ifndef NDEBUG
 // Outside the WebCore namespace for ease of invocation from gdb.
 CORE_EXPORT void showLiveDocumentInstances();
 #endif
 
 #endif  // Document_h