Added WebStateImpl::OnNavigationFinished().

Added WebStateImpl::OnNavigationFinished().

This CL (4 of 7) is a part of large refactoring to create
NavigationContext when navigation was started, not when it is finished.
This refactoring will allow to pass the same instance of
NavigationContext to DidStartNavigation and DidFinishNavigation
(crbug.com/713836). Also this will allow to associate NavigationItem
with NavigationContext object to commit non-latest navigation
(crbug.com/712269).

This CL adds WebStateimpl::OnNavigationFinished which simply calls
WebStateObserver::DidFinishNavigation. This method will be used to
pass NavigationContext (which will be created earlier) to DidFinishNavigation.

BUG=713836, 712269
Review-Url: https://codereview.chromium.org/2833423002
Cr-Commit-Position: refs/heads/master@{#470314}
Committed: https://chromium.googlesource.com/chromium/src/+/1000a00b446127c2c0148fe11b2bf2b8223e85b4

[Eugene But (OOO till 7-30), 2017-04-24 12:14:31]

Description was changed from

==========
Added WebStateImpl::OnNavigationFinished().

This CL (4 of 7) is a part of large refactoring to create NavigationContext
when navigation was started, not when it is finished. This refactoring will
allow to pass the same instance of NavigationContext to DidStartNavigation and
DidFinishNavigation (crbug.com/713836). Also this will allow to associate
NavigationItem with NavigationContext object to commit non-latest navigation
(crbug.com/712269).

BUG=713836, 712269
==========

to

==========
Added WebStateImpl::OnNavigationFinished().

This CL (4 of 7) is a part of large refactoring to create NavigationContext
when navigation was started, not when it is finished. This refactoring will
allow to pass the same instance of NavigationContext to DidStartNavigation and
DidFinishNavigation (crbug.com/713836). Also this will allow to associate
NavigationItem with NavigationContext object to commit non-latest navigation
(crbug.com/712269).

This CL adds WebStateimpl::OnNavigationFinished which simply calls
WebStateObserver::DidFinishNavigation. This method will be used to
pass NavigationContext (which will be created earlier) to DidFinishNavigation.

BUG=713836, 712269
==========

[Eugene But (OOO till 7-30), 2017-04-24 12:52:32]

Description was changed from

==========
Added WebStateImpl::OnNavigationFinished().

This CL (4 of 7) is a part of large refactoring to create NavigationContext
when navigation was started, not when it is finished. This refactoring will
allow to pass the same instance of NavigationContext to DidStartNavigation and
DidFinishNavigation (crbug.com/713836). Also this will allow to associate
NavigationItem with NavigationContext object to commit non-latest navigation
(crbug.com/712269).

This CL adds WebStateimpl::OnNavigationFinished which simply calls
WebStateObserver::DidFinishNavigation. This method will be used to
pass NavigationContext (which will be created earlier) to DidFinishNavigation.

BUG=713836, 712269
==========

to

==========
Added WebStateImpl::OnNavigationFinished().

This CL (4 of 7) is a part of large refactoring to create
NavigationContext when navigation was started, not when it is finished.
This refactoring will allow to pass the same instance of
NavigationContext to DidStartNavigation and DidFinishNavigation
(crbug.com/713836). Also this will allow to associate NavigationItem
with NavigationContext object to commit non-latest navigation
(crbug.com/712269).

This CL adds WebStateimpl::OnNavigationFinished which simply calls
WebStateObserver::DidFinishNavigation. This method will be used to
pass NavigationContext (which will be created earlier) to DidFinishNavigation.

BUG=713836, 712269
==========

[Eugene But (OOO till 7-30), 2017-04-24 14:10:25]

eugenebut@chromium.org changed reviewers:
+ kkhorimoto@chromium.org

[kkhorimoto, 2017-04-25 13:47:41]

lgtm

[Eugene But (OOO till 7-30), 2017-05-09 13:40:01]

The CQ bit was checked by eugenebut@chromium.org

[Eugene But (OOO till 7-30), 2017-05-09 13:40:02]

The patchset sent to the CQ was uploaded after l-g-t-m from
kkhorimoto@chromium.org
Link to the patchset: https://codereview.chromium.org/2833423002/#ps20001
(title: "Rebased")

[commit-bot: I haz the power, 2017-05-09 13:40:26]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-09 13:52:40]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1494337201917140,
"parent_rev": "1073d1c2e24a31bc6c34bbc6a6c198ae5d668086", "commit_rev":
"1000a00b446127c2c0148fe11b2bf2b8223e85b4"}

[commit-bot: I haz the power, 2017-05-09 13:52:54]

Description was changed from

==========
Added WebStateImpl::OnNavigationFinished().

This CL (4 of 7) is a part of large refactoring to create
NavigationContext when navigation was started, not when it is finished.
This refactoring will allow to pass the same instance of
NavigationContext to DidStartNavigation and DidFinishNavigation
(crbug.com/713836). Also this will allow to associate NavigationItem
with NavigationContext object to commit non-latest navigation
(crbug.com/712269).

This CL adds WebStateimpl::OnNavigationFinished which simply calls
WebStateObserver::DidFinishNavigation. This method will be used to
pass NavigationContext (which will be created earlier) to DidFinishNavigation.

BUG=713836, 712269
==========

to

==========
Added WebStateImpl::OnNavigationFinished().

This CL (4 of 7) is a part of large refactoring to create
NavigationContext when navigation was started, not when it is finished.
This refactoring will allow to pass the same instance of
NavigationContext to DidStartNavigation and DidFinishNavigation
(crbug.com/713836). Also this will allow to associate NavigationItem
with NavigationContext object to commit non-latest navigation
(crbug.com/712269).

This CL adds WebStateimpl::OnNavigationFinished which simply calls
WebStateObserver::DidFinishNavigation. This method will be used to
pass NavigationContext (which will be created earlier) to DidFinishNavigation.

BUG=713836, 712269

Review-Url: https://codereview.chromium.org/2833423002
Cr-Commit-Position: refs/heads/master@{#470314}
Committed:
https://chromium.googlesource.com/chromium/src/+/1000a00b446127c2c0148fe11b2b...
==========

[commit-bot: I haz the power, 2017-05-09 13:52:55]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/1000a00b446127c2c0148fe11b2b...