Replace Document::postTask with WebTaskRunner::postTask

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

 /*
  * 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, 2011, 2012 Apple Inc. All
  * rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved.
@@ skipping 367 matching lines @@
   DCHECK(hasEditableStyle(element));
 
   return element.document().frame() && rootEditableElement(element);
 }
 
 uint64_t Document::s_globalTreeVersion = 0;
 
 static bool s_threadedParsingEnabledForTesting = true;
 
 // This doesn't work with non-Document ExecutionContext.
 static void runAutofocusTask(ExecutionContext* context) {

[tzik, 2017/02/09 23:50:20]

null check for |context| is needed here too.

[yuryu, 2017/02/13 03:33:22]

Done.

   Document* document = toDocument(context);
   if (Element* element = document->autofocusElement()) {
     document->setAutofocusElement(0);
     element->focus();
   }
 }
 
 static void recordLoadReasonToHistogram(WouldLoadReason reason) {
   DEFINE_STATIC_LOCAL(
       EnumerationHistogram, unseenFrameHistogram,
@@ skipping 5306 matching lines @@
 
 ResizeObserverController& Document::ensureResizeObserverController() {
   if (!m_resizeObserverController)
     m_resizeObserverController = new ResizeObserverController();
   return *m_resizeObserverController;
 }
 
 static void runAddConsoleMessageTask(MessageSource source,
                                      MessageLevel level,
                                      const String& message,
                                      ExecutionContext* context) {

[tzik, 2017/02/09 23:50:19]

null check for |context| is needed here, since postTask/crossThreadBind doesn't
have the task cancellation.

[yuryu, 2017/02/13 03:33:22]

Done.

   context->addConsoleMessage(ConsoleMessage::create(source, level, message));
 }
 
 void Document::addConsoleMessage(ConsoleMessage* consoleMessage) {
   if (!isContextThread()) {
-    postTask(TaskType::Unthrottled, BLINK_FROM_HERE,
-             createCrossThreadTask(
-                 &runAddConsoleMessageTask, consoleMessage->source(),
-                 consoleMessage->level(), consoleMessage->message()));
+    TaskRunnerHelper::get(TaskType::Unthrottled, this)
+        ->postTask(
+            BLINK_FROM_HERE,
+            crossThreadBind(&runAddConsoleMessageTask, consoleMessage->source(),
+                            consoleMessage->level(), consoleMessage->message(),
+                            wrapCrossThreadWeakPersistent(this)));
     return;
   }
 
   if (!m_frame)
     return;
 
   if (consoleMessage->location()->isUnknown()) {
     // TODO(dgozman): capture correct location at call places instead.
     unsigned lineNumber = 0;
     if (!isInDocumentWrite() && scriptableDocumentParser()) {
@@ skipping 566 matching lines @@
 void Document::setAutofocusElement(Element* element) {
   if (!element) {
     m_autofocusElement = nullptr;
     return;
   }
   if (m_hasAutofocused)
     return;
   m_hasAutofocused = true;
   DCHECK(!m_autofocusElement);
   m_autofocusElement = element;
-  postTask(TaskType::UserInteraction, BLINK_FROM_HERE,
-           createSameThreadTask(&runAutofocusTask));
+  TaskRunnerHelper::get(TaskType::UserInteraction, this)
+      ->postTask(BLINK_FROM_HERE,
+                 WTF::bind(&runAutofocusTask, wrapWeakPersistent(this)));
 }
 
 Element* Document::activeElement() const {
   if (Element* element = adjustedFocusedElement())
     return element;
   return body();
 }
 
 bool Document::hasFocus() const {
   return page() && page()->focusController().isDocumentFocused(*this);
@@ skipping 253 matching lines @@
 }
 
 void showLiveDocumentInstances() {
   WeakDocumentSet& set = liveDocumentSet();
   fprintf(stderr, "There are %u documents currently alive:\n", set.size());
   for (blink::Document* document : set)
     fprintf(stderr, "- Document %p URL: %s\n", document,
             document->url().getString().utf8().data());
 }
 #endif