Create NavigationContext right after the navigation was started.

Create NavigationContext right after the navigation was started.

This CL (7 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 changes the time of NavigationContext creation. Subsequent CLs
will fix:
 - crbug.com/713836 by passing NavigationContext to
      OnProvisionalNavigationStarted
 - crbug.com/712269 by looking up for pending NavigationContext and
      extracting NavigationItem from it

BUG=713836, 712269
Review-Url: https://codereview.chromium.org/2842443002
Cr-Commit-Position: refs/heads/master@{#470687}
Committed: https://chromium.googlesource.com/chromium/src/+/756b09442132dec9b01cbeda19d0a8d69e8dd551

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

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

[Eugene But (OOO till 7-30), 2017-04-26 06:13:35]

Description was changed from

==========
Create NavigationContext right after the navigation was started.

This CL (7 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 changes the time of NavigationContext creation. Subsequent CLs
will fix crbug.com/713836 and crbug.com/712269.

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

to

==========
Create NavigationContext right after the navigation was started.

This CL (7 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 changes the time of NavigationContext creation. Subsequent CLs
will fix:
 - crbug.com/713836 by passing NavigationContext to
      OnProvisionalNavigationStarted
 - crbug.com/712269 by looking up for pending NavigationContext and
      extracting NavigationItem from it

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

[kkhorimoto, 2017-05-04 19:07:15]

lgtm

[Eugene But (OOO till 7-30), 2017-05-10 15:13:06]

The CQ bit was checked by eugenebut@chromium.org

[Eugene But (OOO till 7-30), 2017-05-10 15:13:06]

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/2842443002/#ps80001
(title: "Rebased")

[commit-bot: I haz the power, 2017-05-10 15:13:44]

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-10 15:47:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-10 15:47:34]

Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Eugene But (OOO till 7-30), 2017-05-10 18:31:07]

The CQ bit was checked by eugenebut@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-10 18:33:33]

Dry run: 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-10 19:00:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-10 19:00:47]

Dry run: This issue passed the CQ dry run.

[Eugene But (OOO till 7-30), 2017-05-10 20:13:23]

The CQ bit was checked by eugenebut@chromium.org

[Eugene But (OOO till 7-30), 2017-05-10 20:13:24]

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/2842443002/#ps100001
(title: "Fixed tests")

[commit-bot: I haz the power, 2017-05-10 20:14:29]

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-10 20:28:42]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1494447203815350,
"parent_rev": "d571f3812f6e6031ce0da38c6a667cbabfd31e36", "commit_rev":
"756b09442132dec9b01cbeda19d0a8d69e8dd551"}

[commit-bot: I haz the power, 2017-05-10 20:28:57]

Description was changed from

==========
Create NavigationContext right after the navigation was started.

This CL (7 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 changes the time of NavigationContext creation. Subsequent CLs
will fix:
 - crbug.com/713836 by passing NavigationContext to
      OnProvisionalNavigationStarted
 - crbug.com/712269 by looking up for pending NavigationContext and
      extracting NavigationItem from it

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

to

==========
Create NavigationContext right after the navigation was started.

This CL (7 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 changes the time of NavigationContext creation. Subsequent CLs
will fix:
 - crbug.com/713836 by passing NavigationContext to
      OnProvisionalNavigationStarted
 - crbug.com/712269 by looking up for pending NavigationContext and
      extracting NavigationItem from it

BUG=713836, 712269

Review-Url: https://codereview.chromium.org/2842443002
Cr-Commit-Position: refs/heads/master@{#470687}
Committed:
https://chromium.googlesource.com/chromium/src/+/756b09442132dec9b01cbeda19d0...
==========

[commit-bot: I haz the power, 2017-05-10 20:29:00]

Committed patchset #6 (id:100001) as
https://chromium.googlesource.com/chromium/src/+/756b09442132dec9b01cbeda19d0...