Added -[Tab webState:didCommitNavigationWithDetails:].

Added -[Tab webState:didCommitNavigationWithDetails:].

Moved most of -[Tab webDidStartLoadingURL:shouldUpdateHistory:] to this
new method. |didCommitNavigationWithDetails:| is called right before
|webDidStartLoadingURL:shouldUpdateHistory:|, so this CL should not
change the logic.

|webDidStartLoadingURL:shouldUpdateHistory:| is a part of deprecated
CRWWebDelegate and should be fully removed in a separate CL.

BUG=674991
Review-Url: https://codereview.chromium.org/2643493002
Cr-Commit-Position: refs/heads/master@{#444381}
Committed: https://chromium.googlesource.com/chromium/src/+/4f2c57e57018d7c3b19c877171a062fd1508a0f6

[Eugene But (OOO till 7-30), 2017-01-17 21:37:29]

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

[commit-bot: I haz the power, 2017-01-17 21:37:53]

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-01-17 21:57:29]

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

[commit-bot: I haz the power, 2017-01-17 21:57:29]

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-01-17 23:30:25]

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

[commit-bot: I haz the power, 2017-01-17 23:31:17]

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-01-17 23:44:20]

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

[commit-bot: I haz the power, 2017-01-17 23:44:21]

Dry run: This issue passed the CQ dry run.

[Eugene But (OOO till 7-30), 2017-01-18 00:23:24]

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

[marq (ping after 24h), 2017-01-18 10:04:55]

lgtm

[Eugene But (OOO till 7-30), 2017-01-18 16:11:08]

The CQ bit was checked by eugenebut@chromium.org

[commit-bot: I haz the power, 2017-01-18 16:11:34]

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-01-18 16:15:45]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1484755868299430,
"parent_rev": "921882bd5ee50f1e59288a51f0fd018261f3ffc2", "commit_rev":
"4f2c57e57018d7c3b19c877171a062fd1508a0f6"}

[commit-bot: I haz the power, 2017-01-18 16:16:17]

Description was changed from

==========
Added -[Tab webState:didCommitNavigationWithDetails:].

Moved most of -[Tab webDidStartLoadingURL:shouldUpdateHistory:] to this
new method. |didCommitNavigationWithDetails:| is called right before
|webDidStartLoadingURL:shouldUpdateHistory:|, so this CL should not
change the logic.

|webDidStartLoadingURL:shouldUpdateHistory:| is a part of deprecated
CRWWebDelegate and should be fully removed in a separate CL.

BUG=674991
==========

to

==========
Added -[Tab webState:didCommitNavigationWithDetails:].

Moved most of -[Tab webDidStartLoadingURL:shouldUpdateHistory:] to this
new method. |didCommitNavigationWithDetails:| is called right before
|webDidStartLoadingURL:shouldUpdateHistory:|, so this CL should not
change the logic.

|webDidStartLoadingURL:shouldUpdateHistory:| is a part of deprecated
CRWWebDelegate and should be fully removed in a separate CL.

BUG=674991

Review-Url: https://codereview.chromium.org/2643493002
Cr-Commit-Position: refs/heads/master@{#444381}
Committed:
https://chromium.googlesource.com/chromium/src/+/4f2c57e57018d7c3b19c877171a0...
==========

[commit-bot: I haz the power, 2017-01-18 16:16:18]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/4f2c57e57018d7c3b19c877171a0...