Revert of Add DidSaveScrollOrScaleState flag to prevent restoreScrollPosition...

Revert of Add DidSaveScrollOrScaleState flag to prevent restoreScrollPositionAndViewState restore from default (patchset #19 id:380001 of https://codereview.chromium.org/2628683003/ )

Reason for revert:
Memory regression. https://bugs.chromium.org/p/chromium/issues/detail?id=694569

Original issue's description:
> Add DidSaveScrollOrScaleState flag to prevent restoreScrollPositionAndViewState restore
> from default
> 
> This flaky is caused by |FrameLoader::restoreScrollPositionAndViewState| called
> before |FrameLoader::processFragment| in |checkCompleted| and restore from 
> default. 
> 
> So we add DidSaveScrollOrScaleState flag to prevent restoreScrollPositionAndViewState
> restore from default.
> 
> 
> BUG=672545
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Review-Url: https://codereview.chromium.org/2628683003
> Cr-Commit-Position: refs/heads/master@{#451111}
> Committed: https://chromium.googlesource.com/chromium/src/+/1033be6d096d430b3d1fe6439550448b2374ea4c

TBR=bokan@chromium.org,majidvp@chromium.org,japhet@chromium.org,alexmos@chromium.org,creis@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=672545
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

[chaopeng, 2017-02-22 18:52:27]

The CQ bit was checked by chaopeng@chromium.org

[chaopeng, 2017-02-22 18:52:28]

Created Revert of Add DidSaveScrollOrScaleState flag to prevent
restoreScrollPositionAndViewState restore from default

[commit-bot: I haz the power, 2017-02-22 18:53:49]

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

[chaopeng, 2017-02-22 19:00:26]

The CQ bit was unchecked by chaopeng@chromium.org

[Charlie Reis, 2017-02-22 19:01:31]

Is this safe to land?  I'm worried this will break session restore for anyone
who upgraded their serialization version from 23 to 24 already.

[bokan, 2017-02-22 19:04:53]

On 2017/02/22 19:01:31, Charlie Reis wrote:
> Is this safe to land?  I'm worried this will break session restore for anyone
> who upgraded their serialization version from 23 to 24 already.

Doh! Yeah, I hadn't thought of that. Lets scratch the revert and try to repro
locally.

[chaopeng, 2017-04-23 02:22:36]

Description was changed from

==========
Revert of Add DidSaveScrollOrScaleState flag to prevent
restoreScrollPositionAndViewState restore from default (patchset #19 id:380001
of https://codereview.chromium.org/2628683003/ )

Reason for revert:
Memory regression. https://bugs.chromium.org/p/chromium/issues/detail?id=694569

Original issue's description:
> Add DidSaveScrollOrScaleState flag to prevent
restoreScrollPositionAndViewState restore
> from default
> 
> This flaky is caused by |FrameLoader::restoreScrollPositionAndViewState|
called
> before |FrameLoader::processFragment| in |checkCompleted| and restore from 
> default. 
> 
> So we add DidSaveScrollOrScaleState flag to prevent
restoreScrollPositionAndViewState
> restore from default.
> 
> 
> BUG=672545
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Review-Url: https://codereview.chromium.org/2628683003
> Cr-Commit-Position: refs/heads/master@{#451111}
> Committed:
https://chromium.googlesource.com/chromium/src/+/1033be6d096d430b3d1fe6439550...

TBR=bokan@chromium.org,majidvp@chromium.org,japhet@chromium.org,alexmos@chrom...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=672545
==========

to

==========
Revert of Add DidSaveScrollOrScaleState flag to prevent
restoreScrollPositionAndViewState restore from default (patchset #19 id:380001
of https://codereview.chromium.org/2628683003/ )

Reason for revert:
Memory regression. https://bugs.chromium.org/p/chromium/issues/detail?id=694569

Original issue's description:
> Add DidSaveScrollOrScaleState flag to prevent
restoreScrollPositionAndViewState restore
> from default
> 
> This flaky is caused by |FrameLoader::restoreScrollPositionAndViewState|
called
> before |FrameLoader::processFragment| in |checkCompleted| and restore from 
> default. 
> 
> So we add DidSaveScrollOrScaleState flag to prevent
restoreScrollPositionAndViewState
> restore from default.
> 
> 
> BUG=672545
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Review-Url: https://codereview.chromium.org/2628683003
> Cr-Commit-Position: refs/heads/master@{#451111}
> Committed:
https://chromium.googlesource.com/chromium/src/+/1033be6d096d430b3d1fe6439550...

TBR=bokan@chromium.org,majidvp@chromium.org,japhet@chromium.org,alexmos@chrom...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=672545
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[chaopeng, 2017-05-08 18:03:30]

Description was changed from

==========
Revert of Add DidSaveScrollOrScaleState flag to prevent
restoreScrollPositionAndViewState restore from default (patchset #19 id:380001
of https://codereview.chromium.org/2628683003/ )

Reason for revert:
Memory regression. https://bugs.chromium.org/p/chromium/issues/detail?id=694569

Original issue's description:
> Add DidSaveScrollOrScaleState flag to prevent
restoreScrollPositionAndViewState restore
> from default
> 
> This flaky is caused by |FrameLoader::restoreScrollPositionAndViewState|
called
> before |FrameLoader::processFragment| in |checkCompleted| and restore from 
> default. 
> 
> So we add DidSaveScrollOrScaleState flag to prevent
restoreScrollPositionAndViewState
> restore from default.
> 
> 
> BUG=672545
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Review-Url: https://codereview.chromium.org/2628683003
> Cr-Commit-Position: refs/heads/master@{#451111}
> Committed:
https://chromium.googlesource.com/chromium/src/+/1033be6d096d430b3d1fe6439550...

TBR=bokan@chromium.org,majidvp@chromium.org,japhet@chromium.org,alexmos@chrom...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=672545
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Revert of Add DidSaveScrollOrScaleState flag to prevent
restoreScrollPositionAndViewState restore from default (patchset #19 id:380001
of https://codereview.chromium.org/2628683003/ )

Reason for revert:
Memory regression. https://bugs.chromium.org/p/chromium/issues/detail?id=694569

Original issue's description:
> Add DidSaveScrollOrScaleState flag to prevent
restoreScrollPositionAndViewState restore
> from default
> 
> This flaky is caused by |FrameLoader::restoreScrollPositionAndViewState|
called
> before |FrameLoader::processFragment| in |checkCompleted| and restore from 
> default. 
> 
> So we add DidSaveScrollOrScaleState flag to prevent
restoreScrollPositionAndViewState
> restore from default.
> 
> 
> BUG=672545
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Review-Url: https://codereview.chromium.org/2628683003
> Cr-Commit-Position: refs/heads/master@{#451111}
> Committed:
https://chromium.googlesource.com/chromium/src/+/1033be6d096d430b3d1fe6439550...

TBR=bokan@chromium.org,majidvp@chromium.org,japhet@chromium.org,alexmos@chrom...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=672545
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========