HistoryController shouldn't call confusingAndOftenMisusedAttached

Source/core/loader/HistoryController.cpp

Index: Source/core/loader/HistoryController.cpp
diff --git a/Source/core/loader/HistoryController.cpp b/Source/core/loader/HistoryController.cpp
index f6d2cec7ca5ac9b565de504e2e11d680d146dfb5..8b2360ec92b4fc89ad5271367decce4b499cc962 100644
--- a/Source/core/loader/HistoryController.cpp
+++ b/Source/core/loader/HistoryController.cpp
@@ -123,7 +123,7 @@ void HistoryController::saveDocumentAndScrollState()
     Document* document = m_frame->document();
     ASSERT(document);
 
-    if (m_currentItem->isCurrentDocument(document) && document->confusingAndOftenMisusedAttached()) {
+    if (m_currentItem->isCurrentDocument(document) && document->renderer()) {

[esprehn, 2013/09/27 18:23:45]

I usually use renderView() since it's more clear, we should probably make
Document::renderer private.

[abarth-chromium, 2013/09/27 18:26:13]

Ok.  That will require me to pull in another header (because it's defined in
RenderView.h), but that's fine.

         LOG(Loading, "WebCoreLoading %s: saving form state to %p", m_frame->tree()->uniqueName().string().utf8().data(), m_currentItem.get());
         m_currentItem->setDocumentState(document->formElementsState());
     }