Call DOMWindow::finishedLoading from FrameLoader::checkLoadCompleteForThisFrame

Call DOMWindow::finishedLoading from FrameLoader::checkLoadCompleteForThisFrame

Currently, calling DOMWindow::finishedLoading is called from DocumentLoader's
checkLoadComplete(), which is called from several different places. Just call
finishedLoading() once, from FrameLoader::checkLoadCompleteForThisFrame.

This also moves finishedLoading() to immediately after sending a didStopLoading
to the embedder. Because DOMWindow::finishedLoading's primary purpose is to
begin a page print, this ensures that print is not called until the embedder
has been notified that the load has stopped.

BUG=none, though the reorder might help resolve 365523

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=172896

[Nate Chapin, 2014-04-25 20:15:10]

There's a failing browser test that depends on the ordering of stop and print,
so https://codereview.chromium.org/250953005/ will need to land before this.

[abarth-chromium, 2014-04-25 20:57:11]

If this works, LGTM!

[Nate Chapin, 2014-04-28 19:19:25]

The CQ bit was checked by japhet@chromium.org

[Nate Chapin, 2014-04-28 19:19:49]

The CQ bit was unchecked by japhet@chromium.org

[commit-bot: I haz the power, 2014-04-28 19:19:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/japhet@chromium.org/252383010/1

[Nate Chapin, 2014-04-28 19:20:14]

The CQ bit was unchecked by japhet@chromium.org

[Nate Chapin, 2014-04-29 20:10:09]

The CQ bit was checked by japhet@chromium.org

[commit-bot: I haz the power, 2014-04-29 20:26:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/japhet@chromium.org/252383010/1

[commit-bot: I haz the power, 2014-04-30 03:38:45]

Change committed as 172896