Revert of Fixed title updating for back forward navigation.

Revert of Fixed title updating for back forward navigation. (patchset #3 id:40001 of https://codereview.chromium.org/2685803002/ )

Reason for revert:
Breaks -[TabHistoryPopupControllerTestCase testTabHistoryMenu] tab_history_popup_controller_egtest.mm downstream, because New Tab in history popup (when long pressed on back) displays "chrome://newtab" instead of "New Tab".

I actually don't know why this test is not ran upstream, but I have to revert, sorry!

Original issue's description:
> Fixed title updating for back forward navigation.
> 
> Chrome relies on KVO compliant "title" property to subscribe for
> title updates. If this KVO change happens during then navigation
> it is unclear if title was changes for the previous page, or the
> navigation was committed and title was changed for the new page.
> 
> So if there is a navigation in progress WebController should ignore
> KVO change, but it should always attempt to update title when
> navigation is committed.
> 
> BUG=688047, 677356
> 
> Review-Url: https://codereview.chromium.org/2685803002
> Cr-Commit-Position: refs/heads/master@{#449527}
> Committed: https://chromium.googlesource.com/chromium/src/+/487e4cfd53bac5944dad49cbce3b939a0c7bf035

TBR=kkhorimoto@chromium.org,eugenebut@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=688047, 677356
Review-Url: https://codereview.chromium.org/2684343003
Cr-Commit-Position: refs/heads/master@{#449595}
Committed: https://chromium.googlesource.com/chromium/src/+/69378e0417fa4dbcae4dc863e5b46b68d71ac0be

[stkhapugin, 2017-02-10 12:09:00]

The CQ bit was checked by stkhapugin@chromium.org

[stkhapugin, 2017-02-10 12:09:01]

Created Revert of Fixed title updating for back forward navigation.

[commit-bot: I haz the power, 2017-02-10 12:09:21]

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

[stkhapugin, 2017-02-10 12:09:38]

Description was changed from

==========
Revert of Fixed title updating for back forward navigation. (patchset #3
id:40001 of https://codereview.chromium.org/2685803002/ )

Reason for revert:
Breaks -[TabHistoryPopupControllerTestCase testTabHistoryMenu]
tab_history_popup_controller_egtest.mm downstream, because New Tab in history
popup (when long pressed on back) displays "chrome://newtab" instead of "New
Tab".

I actually don't know why this test is not ran upstream, but I have to revert,
sorry!

Original issue's description:
> Fixed title updating for back forward navigation.
> 
> Chrome relies on KVO compliant "title" property to subscribe for
> title updates. If this KVO change happens during then navigation
> it is unclear if title was changes for the previous page, or the
> navigation was committed and title was changed for the new page.
> 
> So if there is a navigation in progress WebController should ignore
> KVO change, but it should always attempt to update title when
> navigation is committed.
> 
> BUG=688047, 677356
> 
> Review-Url: https://codereview.chromium.org/2685803002
> Cr-Commit-Position: refs/heads/master@{#449527}
> Committed:
https://chromium.googlesource.com/chromium/src/+/487e4cfd53bac5944dad49cbce3b...

TBR=kkhorimoto@chromium.org,eugenebut@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=688047, 677356
==========

to

==========
Revert of Fixed title updating for back forward navigation. (patchset #3
id:40001 of https://codereview.chromium.org/2685803002/ )

Reason for revert:
Breaks -[TabHistoryPopupControllerTestCase testTabHistoryMenu]
tab_history_popup_controller_egtest.mm downstream, because New Tab in history
 popup (when long pressed on back) displays "chrome://newtab" instead of "New
 Tab".

I actually don't know why this test is not ran upstream, but I have to revert,
sorry!

Original issue's description:
> Fixed title updating for back forward navigation.
> 
> Chrome relies on KVO compliant "title" property to subscribe for
> title updates. If this KVO change happens during then navigation
> it is unclear if title was changes for the previous page, or the
> navigation was committed and title was changed for the new page.
> 
> So if there is a navigation in progress WebController should ignore
> KVO change, but it should always attempt to update title when
> navigation is committed.
> 
> BUG=688047, 677356
> 
> Review-Url: https://codereview.chromium.org/2685803002
> Cr-Commit-Position: refs/heads/master@{#449527}
> Committed:
https://chromium.googlesource.com/chromium/src/+/487e4cfd53bac5944dad49cbce3b...

TBR=kkhorimoto@chromium.org,eugenebut@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=688047, 677356
==========

[commit-bot: I haz the power, 2017-02-10 12:10:13]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486728540573200, "parent_rev":
"33e953c922b7dcfa88418ae62d098d6d3e869535", "commit_rev":
"69378e0417fa4dbcae4dc863e5b46b68d71ac0be"}

[commit-bot: I haz the power, 2017-02-10 12:10:43]

Description was changed from

==========
Revert of Fixed title updating for back forward navigation. (patchset #3
id:40001 of https://codereview.chromium.org/2685803002/ )

Reason for revert:
Breaks -[TabHistoryPopupControllerTestCase testTabHistoryMenu]
tab_history_popup_controller_egtest.mm downstream, because New Tab in history
 popup (when long pressed on back) displays "chrome://newtab" instead of "New
 Tab".

I actually don't know why this test is not ran upstream, but I have to revert,
sorry!

Original issue's description:
> Fixed title updating for back forward navigation.
> 
> Chrome relies on KVO compliant "title" property to subscribe for
> title updates. If this KVO change happens during then navigation
> it is unclear if title was changes for the previous page, or the
> navigation was committed and title was changed for the new page.
> 
> So if there is a navigation in progress WebController should ignore
> KVO change, but it should always attempt to update title when
> navigation is committed.
> 
> BUG=688047, 677356
> 
> Review-Url: https://codereview.chromium.org/2685803002
> Cr-Commit-Position: refs/heads/master@{#449527}
> Committed:
https://chromium.googlesource.com/chromium/src/+/487e4cfd53bac5944dad49cbce3b...

TBR=kkhorimoto@chromium.org,eugenebut@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=688047, 677356
==========

to

==========
Revert of Fixed title updating for back forward navigation. (patchset #3
id:40001 of https://codereview.chromium.org/2685803002/ )

Reason for revert:
Breaks -[TabHistoryPopupControllerTestCase testTabHistoryMenu]
tab_history_popup_controller_egtest.mm downstream, because New Tab in history
popup (when long pressed on back) displays "chrome://newtab" instead of "New
Tab".

I actually don't know why this test is not ran upstream, but I have to revert,
sorry!

Original issue's description:
> Fixed title updating for back forward navigation.
> 
> Chrome relies on KVO compliant "title" property to subscribe for
> title updates. If this KVO change happens during then navigation
> it is unclear if title was changes for the previous page, or the
> navigation was committed and title was changed for the new page.
> 
> So if there is a navigation in progress WebController should ignore
> KVO change, but it should always attempt to update title when
> navigation is committed.
> 
> BUG=688047, 677356
> 
> Review-Url: https://codereview.chromium.org/2685803002
> Cr-Commit-Position: refs/heads/master@{#449527}
> Committed:
https://chromium.googlesource.com/chromium/src/+/487e4cfd53bac5944dad49cbce3b...

TBR=kkhorimoto@chromium.org,eugenebut@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=688047, 677356

Review-Url: https://codereview.chromium.org/2684343003
Cr-Commit-Position: refs/heads/master@{#449595}
Committed:
https://chromium.googlesource.com/chromium/src/+/69378e0417fa4dbcae4dc863e5b4...
==========

[commit-bot: I haz the power, 2017-02-10 12:10:44]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/69378e0417fa4dbcae4dc863e5b4...

[Eugene But (OOO till 7-30), 2017-02-10 18:27:06]

lgtm Sorry for breaking this