Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

This callback is replaced by WebStateObserver::DidFinishNavigation.

Call OnNavigationCommitted after |webPageChanged|, because 
NavigationItem is committed inside |webPageChanged|. 
UpdateHttpResponseHeaders() should still be called before
SetContentsMimeType(), so |UpdateHttpResponseHeaders| was moved
out of |OnNavigationCommitted|.

BUG=692331, 698410
Review-Url: https://codereview.chromium.org/2724383003
Cr-Commit-Position: refs/heads/master@{#455167}
Committed: https://chromium.googlesource.com/chromium/src/+/9b8867c5b4b08dd484fb631fadacb6d247cdc7cf

[Eugene But (OOO till 7-30), 2017-03-02 19:33:08]

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

[commit-bot: I haz the power, 2017-03-02 19:33:44]

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-03-02 19:53:33]

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

[commit-bot: I haz the power, 2017-03-02 19:53:34]

Dry run: This issue passed the CQ dry run.

[Eugene But (OOO till 7-30), 2017-03-02 23:26:53]

Description was changed from

==========
Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

This callback is relaced by WebStateObserver::DidFinishNavigation.

BUG=692331
==========

to

==========
Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

This callback is relaced by WebStateObserver::DidFinishNavigation.
Call OnNavigationCommitted after |webPageChanged|, because 
NavigationItem is committed inside |webPageChanged|.

BUG=692331
==========

[Eugene But (OOO till 7-30), 2017-03-03 01:58:46]

Description was changed from

==========
Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

This callback is relaced by WebStateObserver::DidFinishNavigation.
Call OnNavigationCommitted after |webPageChanged|, because 
NavigationItem is committed inside |webPageChanged|.

BUG=692331
==========

to

==========
Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

This callback is relaced by WebStateObserver::DidFinishNavigation.
Call OnNavigationCommitted after |webPageChanged|, because 
NavigationItem is committed inside |webPageChanged|.
OnSamePageNavigation is now called inside completion handler, when
state changes are finished.

BUG=692331
==========

[Eugene But (OOO till 7-30), 2017-03-03 02:31:05]

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

[Eugene But (OOO till 7-30), 2017-03-03 05:21:19]

Description was changed from

==========
Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

This callback is relaced by WebStateObserver::DidFinishNavigation.
Call OnNavigationCommitted after |webPageChanged|, because 
NavigationItem is committed inside |webPageChanged|.
OnSamePageNavigation is now called inside completion handler, when
state changes are finished.

BUG=692331
==========

to

==========
Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

This callback is relaced by WebStateObserver::DidFinishNavigation.
Call OnNavigationCommitted after |webPageChanged|, because 
NavigationItem is committed inside |webPageChanged|.

BUG=692331
==========

[Eugene But (OOO till 7-30), 2017-03-03 05:21:28]

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

[commit-bot: I haz the power, 2017-03-03 05:21:45]

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-03-03 06:13:07]

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

[commit-bot: I haz the power, 2017-03-03 06:13:08]

Dry run: 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-03-04 00:19:58]

Description was changed from

==========
Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

This callback is relaced by WebStateObserver::DidFinishNavigation.
Call OnNavigationCommitted after |webPageChanged|, because 
NavigationItem is committed inside |webPageChanged|.

BUG=692331
==========

to

==========
Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

This callback is relaced by WebStateObserver::DidFinishNavigation.

Call OnNavigationCommitted after |webPageChanged|, because 
NavigationItem is committed inside |webPageChanged|. 
UpdateHttpResponseHeaders() should still be called before
SetContentsMimeType(), so |UpdateHttpResponseHeaders| was moved
out of |OnNavigationCommitted|.

BUG=692331
==========

[Eugene But (OOO till 7-30), 2017-03-04 00:22:59]

Description was changed from

==========
Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

This callback is relaced by WebStateObserver::DidFinishNavigation.

Call OnNavigationCommitted after |webPageChanged|, because 
NavigationItem is committed inside |webPageChanged|. 
UpdateHttpResponseHeaders() should still be called before
SetContentsMimeType(), so |UpdateHttpResponseHeaders| was moved
out of |OnNavigationCommitted|.

BUG=692331
==========

to

==========
Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

This callback is replaced by WebStateObserver::DidFinishNavigation.

Call OnNavigationCommitted after |webPageChanged|, because 
NavigationItem is committed inside |webPageChanged|. 
UpdateHttpResponseHeaders() should still be called before
SetContentsMimeType(), so |UpdateHttpResponseHeaders| was moved
out of |OnNavigationCommitted|.

BUG=692331
==========

[Eugene But (OOO till 7-30), 2017-03-04 00:23:40]

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

[commit-bot: I haz the power, 2017-03-04 00:24:08]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Eugene But (OOO till 7-30), 2017-03-04 00:34:41]

Description was changed from

==========
Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

This callback is replaced by WebStateObserver::DidFinishNavigation.

Call OnNavigationCommitted after |webPageChanged|, because 
NavigationItem is committed inside |webPageChanged|. 
UpdateHttpResponseHeaders() should still be called before
SetContentsMimeType(), so |UpdateHttpResponseHeaders| was moved
out of |OnNavigationCommitted|.

BUG=692331
==========

to

==========
Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

This callback is replaced by WebStateObserver::DidFinishNavigation.

Call OnNavigationCommitted after |webPageChanged|, because 
NavigationItem is committed inside |webPageChanged|. 
UpdateHttpResponseHeaders() should still be called before
SetContentsMimeType(), so |UpdateHttpResponseHeaders| was moved
out of |OnNavigationCommitted|.

BUG=692331,698410
==========

[commit-bot: I haz the power, 2017-03-04 00:54:35]

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

[commit-bot: I haz the power, 2017-03-04 00:54:36]

Dry run: 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-03-04 01:14:26]

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

[commit-bot: I haz the power, 2017-03-04 01:14:41]

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-03-04 02:13:30]

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

[commit-bot: I haz the power, 2017-03-04 02:13:31]

Dry run: This issue passed the CQ dry run.

[Eugene But (OOO till 7-30), 2017-03-07 17:08:25]

Kurt?

[kkhorimoto, 2017-03-07 18:43:08]

lgtm

[Eugene But (OOO till 7-30), 2017-03-07 18:46:41]

The CQ bit was checked by eugenebut@chromium.org

[commit-bot: I haz the power, 2017-03-07 18:47:33]

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-03-07 19:06:37]

CQ is committing da patch.

Bot data: {"patchset_id": 110001, "attempt_start_ts": 1488912401692320,
"parent_rev": "be26050c964a17ced5964a2508461ec780048a33", "commit_rev":
"9b8867c5b4b08dd484fb631fadacb6d247cdc7cf"}

[commit-bot: I haz the power, 2017-03-07 19:07:34]

Description was changed from

==========
Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

This callback is replaced by WebStateObserver::DidFinishNavigation.

Call OnNavigationCommitted after |webPageChanged|, because 
NavigationItem is committed inside |webPageChanged|. 
UpdateHttpResponseHeaders() should still be called before
SetContentsMimeType(), so |UpdateHttpResponseHeaders| was moved
out of |OnNavigationCommitted|.

BUG=692331,698410
==========

to

==========
Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

This callback is replaced by WebStateObserver::DidFinishNavigation.

Call OnNavigationCommitted after |webPageChanged|, because 
NavigationItem is committed inside |webPageChanged|. 
UpdateHttpResponseHeaders() should still be called before
SetContentsMimeType(), so |UpdateHttpResponseHeaders| was moved
out of |OnNavigationCommitted|.

BUG=692331,698410

Review-Url: https://codereview.chromium.org/2724383003
Cr-Commit-Position: refs/heads/master@{#455167}
Committed:
https://chromium.googlesource.com/chromium/src/+/9b8867c5b4b08dd484fb631fadac...
==========

[commit-bot: I haz the power, 2017-03-07 19:07:35]

Committed patchset #7 (id:110001) as
https://chromium.googlesource.com/chromium/src/+/9b8867c5b4b08dd484fb631fadac...