Add an empty boolean to WebFrameClient::didFinishDocumentLoad

Source/web/FrameLoaderClientImpl.cpp

Index: Source/web/FrameLoaderClientImpl.cpp
diff --git a/Source/web/FrameLoaderClientImpl.cpp b/Source/web/FrameLoaderClientImpl.cpp
index 614a692b7d127acfe5676488cd9c9be0bb2b9dfa..d0e3d1f4ebb686a5becc7a986a2dbe0f97c28c81 100644
--- a/Source/web/FrameLoaderClientImpl.cpp
+++ b/Source/web/FrameLoaderClientImpl.cpp
@@ -373,10 +373,10 @@ void FrameLoaderClientImpl::dispatchDidFinishLoading(DocumentLoader* loader,
         m_webFrame->client()->didFinishResourceLoad(m_webFrame, identifier);
 }
 
-void FrameLoaderClientImpl::dispatchDidFinishDocumentLoad()
+void FrameLoaderClientImpl::dispatchDidFinishDocumentLoad(bool empty)

[dcheng, 2015/07/13 20:53:56]

I feel like empty isn't quite descriptive enough. An enum (though that's kind of
annoying to plumb from core -> web -> content) or a name like emptyResponseBody
might help.

[Nate Chapin, 2015/07/13 21:13:28]

I changed the chromium-side to use the name document_is_empty. Is the
blink-styled equivalent OK?

[dcheng, 2015/07/13 21:23:22]

Yes, that seems better.

 {
     if (m_webFrame->client())
-        m_webFrame->client()->didFinishDocumentLoad(m_webFrame);
+        m_webFrame->client()->didFinishDocumentLoad(m_webFrame, empty);
 }
 
 void FrameLoaderClientImpl::dispatchDidLoadResourceFromMemoryCache(const ResourceRequest& request, const ResourceResponse& response)