Move TopDocumentRootScrollerController to a separate object on FrameHost

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 739e80bde553a947bad6b7e5b611c1b2ac21e700..e5eabb105225708b90801c11f6199cccca7ba187 100644
--- a/third_party/WebKit/Source/core/dom/Document.cpp
+++ b/third_party/WebKit/Source/core/dom/Document.cpp
@@ -491,11 +491,7 @@ Document::Document(const DocumentInit& initializer, DocumentClassFlags documentC
         m_fetcher = ResourceFetcher::create(nullptr);
     }
 
-    // TODO(bokan): This will probably blow up if we don't have an m_frame here
-    // since we'll assume a child RootScrollerController. crbug.com/505516.
-    m_rootScrollerController = isInMainFrame()
-        ? TopDocumentRootScrollerController::create(*this)
-        : RootScrollerController::create(*this);
+    m_rootScrollerController = RootScrollerController::create(*this);
 
     // We depend on the url getting immediately set in subframes, but we
     // also depend on the url NOT getting immediately set in opened windows.
@@ -2164,9 +2160,6 @@ void Document::initialize()
 
     if (view())
         view()->didAttachDocument();
-
-    // Needs to be called after view()->didAttachDocument().
-    m_rootScrollerController->didAttachDocument();
 }
 
 void Document::shutdown()