[Reland] Post loading tasks on the appropriate WebFrameScheduler's queue

third_party/WebKit/Source/core/loader/FrameLoader.cpp

Index: third_party/WebKit/Source/core/loader/FrameLoader.cpp
diff --git a/third_party/WebKit/Source/core/loader/FrameLoader.cpp b/third_party/WebKit/Source/core/loader/FrameLoader.cpp
index 05fe06764a622b43a0f3a1bb6f29cff79696e1db..c3abf161d2a3758776e9294a4b8f6bbc736570c8 100644
--- a/third_party/WebKit/Source/core/loader/FrameLoader.cpp
+++ b/third_party/WebKit/Source/core/loader/FrameLoader.cpp
@@ -1072,6 +1072,7 @@ bool FrameLoader::prepareForCommit()
     if (m_frame->document())
         m_frame->document()->detach();
     m_documentLoader = m_provisionalDocumentLoader.release();
+    m_frame->updateFrameSecurityOrigin();

[dcheng, 2015/11/26 19:15:24]

Why was this line added? Why isn't Document::updateSecurityOrigin() good enough?
After all, that's where the SecurityOrigin actually gets set.

[alex clarke (OOO till 29th), 2015/11/27 10:18:00]

This line was added before the one in Document::updateSecurityOrigin.  It's
entirely possible it's not actually needed now.  

We noticed there where situations where queues didn't have the right origin in
tracing which was fixed by adding code in Document::updateSecurityOrigin, it
didn't occur to us to try and remove this line.

 
     return true;
 }