Move some Document timers to frame-specific task runners.

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 458 matching lines @@
       m_useSecureKeyboardEntryWhenActive(false),
       m_documentClasses(documentClasses),
       m_isViewSource(false),
       m_sawElementsInKnownNamespaces(false),
       m_isSrcdocDocument(false),
       m_isMobileDocument(false),
       m_layoutView(0),
       m_contextDocument(initializer.contextDocument()),
       m_hasFullscreenSupplement(false),
       m_loadEventDelayCount(0),
-      m_loadEventDelayTimer(this, &Document::loadEventDelayTimerFired),
-      m_pluginLoadingTimer(this, &Document::pluginLoadingTimerFired),
+      m_loadEventDelayTimer(TaskType::Networking,
+                            this,

[dcheng, 2016/10/19 00:08:39]

This look a bit weird... but I don't have better ideas atm.

+                            this,
+                            &Document::loadEventDelayTimerFired),
+      m_pluginLoadingTimer(TaskType::Embed,

[dcheng, 2016/10/19 00:08:38]

I'm not really sure if this is the right task queue, any thoughts?

(The spec doesn't talk about this task source a lot, except to mention that this
task source is used to run the embed setup steps.... which is sort of plugin
loading? Though in this case, it's just used to force a layout update...)

[haraken, 2016/10/19 03:07:47]

Maybe this is not a speced task? If so we could use Task::Internal, because it's
just internal implementation details to force a layout update.

[dcheng, 2016/10/19 03:47:44]

Done.

+                           this,
+                           this,
+                           &Document::pluginLoadingTimerFired),
       m_documentTiming(*this),
       m_writeRecursionIsTooDeep(false),
       m_writeRecursionDepth(0),
       m_taskRunner(MainThreadTaskRunner::create(this)),
       m_registrationContext(initializer.registrationContext(this)),
       m_elementDataCacheClearTimer(this,
                                    &Document::elementDataCacheClearTimerFired),
       m_timeline(DocumentTimeline::create(this)),
       m_compositorPendingAnimations(new CompositorPendingAnimations()),
       m_templateDocumentHost(nullptr),
@@ skipping 5906 matching lines @@
 }
 
 void showLiveDocumentInstances() {
   WeakDocumentSet& set = liveDocumentSet();
   fprintf(stderr, "There are %u documents currently alive:\n", set.size());
   for (Document* document : set)
     fprintf(stderr, "- Document %p URL: %s\n", document,
             document->url().getString().utf8().data());
 }
 #endif