Commit 24408eac authored by sky@chromium.org's avatar sky@chromium.org

Adds RenderFrameObserver::DidFinishDocumentLoad

No doubt we can remove RenderViewObserver::DidFinishDocumentLoad(),
but that can be done separately (there is already a TODO for it).

BUG=none
TEST=none
R=darin@chromium.org

Review URL: https://codereview.chromium.org/212793004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@260119 0039d316-1c4b-4281-b951-d872f2087c98
parent 08b47caa
......@@ -43,6 +43,7 @@ class CONTENT_EXPORT RenderFrameObserver : public IPC::Listener,
virtual void DidStartProvisionalLoad() {}
virtual void DidFailProvisionalLoad(const blink::WebURLError& error) {}
virtual void DidFinishLoad() {}
virtual void DidFinishDocumentLoad() {}
virtual void WillReleaseScriptContext(v8::Handle<v8::Context> context,
int world_id) {}
virtual void DidClearWindowObject(int world_id) {}
......
......@@ -1776,6 +1776,8 @@ void RenderFrameImpl::didFinishDocumentLoad(blink::WebFrame* frame) {
// TODO(nasko): Remove once we have RenderFrameObserver for this method.
render_view_->didFinishDocumentLoad(frame);
FOR_EACH_OBSERVER(RenderFrameObserver, observers_, DidFinishDocumentLoad());
// Check whether we have new encoding name.
render_view_->UpdateEncoding(frame, frame->view()->pageEncoding().utf8());
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment