Move some Document timers to frame-specific task runners.

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

Index: third_party/WebKit/Source/core/dom/Document.cpp
diff --git a/third_party/WebKit/Source/core/dom/Document.cpp b/third_party/WebKit/Source/core/dom/Document.cpp
index 88e8ddba1d5c4b64bf97c45c03775722e611459a..62bafdaad655774ef1ca22410931940195abdcd8 100644
--- a/third_party/WebKit/Source/core/dom/Document.cpp
+++ b/third_party/WebKit/Source/core/dom/Document.cpp
@@ -476,8 +476,14 @@ Document::Document(const DocumentInit& initializer,
       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),