Fix SSL indicator missing when doing fast back/forward navigations.

Fix SSL indicator missing when doing fast back/forward navigations.

This regressed in r415977, which was fixing the session restore case where a serialized NavigationEntry wouldn't have the SSLStatus. The race in the non-restore case happens because a previous DidStopLoading IPC clears the new NavigationHandle which does have the SSLStatus. In this case, the fix is to only copy the SSLStatus specifically in the restore case.

BUG=727892
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
R=estark@chromium.org

Review-Url: https://codereview.chromium.org/2926803002 .
Cr-Original-Commit-Position: refs/heads/master@{#477837}
Review-Url: https://codereview.chromium.org/2934493002 .
Cr-Commit-Position: refs/branch-heads/3112@{#282}
Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}
Committed: https://chromium.googlesource.com/chromium/src/+/c89b46b3ed1f0f084ff3213750c688c8154e62a5

[jam, 2017-06-09 15:50:51]

Description was changed from

==========
Fix SSL indicator missing when doing fast back/forward navigations.

This regressed in r415977, which was fixing the session restore case where a
serialized NavigationEntry wouldn't have the SSLStatus. The race in the
non-restore case happens because a previous DidStopLoading IPC clears the new
NavigationHandle which does have the SSLStatus. In this case, the fix is to only
copy the SSLStatus specifically in the restore case.

BUG=727892
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
R=estark@chromium.org

Review-Url: https://codereview.chromium.org/2926803002 .
Cr-Commit-Position: refs/heads/master@{#477837}
(cherry picked from commit db06e65dcd8da3f5b8cab60ecb32e9ba89c1598c)
==========

to

==========
Fix SSL indicator missing when doing fast back/forward navigations.

This regressed in r415977, which was fixing the session restore case where a
serialized NavigationEntry wouldn't have the SSLStatus. The race in the
non-restore case happens because a previous DidStopLoading IPC clears the new
NavigationHandle which does have the SSLStatus. In this case, the fix is to only
copy the SSLStatus specifically in the restore case.

BUG=727892
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
R=estark@chromium.org

Review-Url: https://codereview.chromium.org/2926803002 .
Cr-Original-Commit-Position: refs/heads/master@{#477837}
Review-Url: https://codereview.chromium.org/2934493002 .
Cr-Commit-Position: refs/branch-heads/3112@{#282}
Cr-Branched-From:
b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}
Committed:
https://chromium.googlesource.com/chromium/src/+/c89b46b3ed1f0f084ff3213750c6...
==========

[jam, 2017-06-09 15:50:53]

Committed patchset #1 (id:1) manually as
c89b46b3ed1f0f084ff3213750c688c8154e62a5.