Do not pass scrollPosition to updateScrollbars at every callsite

Do not pass scrollPosition to updateScrollbars at every callsite

More followup cleanup to the scroll position code. This is a minor
refactoring to push the scroll position double concent into
updateScrollbars since it does not need to be plumbed at every callsite.
scrollPositionDouble has some subtlety (see comment above it) which is
easy to mess up.

Committed: https://crrev.com/d12c2b3ab25398ae385564c7506f0e4b598169eb
Cr-Commit-Position: refs/heads/master@{#380478}

[pdr., 2016-03-03 21:35:05]

pdr@chromium.org changed reviewers:
+ chrishtr@chromium.org

[pdr., 2016-03-03 21:35:29]

Last patch for this messy code. Just a simple cleanup.

[pdr., 2016-03-03 21:35:38]

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

[commit-bot: I haz the power, 2016-03-03 21:35:55]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1758413002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1758413002/1

[commit-bot: I haz the power, 2016-03-03 23:12:13]

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

[commit-bot: I haz the power, 2016-03-03 23:12:13]

Dry run: This issue passed the CQ dry run.

[chrishtr, 2016-03-10 17:07:30]

The CQ bit was checked by chrishtr@chromium.org

[chrishtr, 2016-03-10 17:07:31]

lgtm

[commit-bot: I haz the power, 2016-03-10 17:07:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1758413002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1758413002/1

[commit-bot: I haz the power, 2016-03-10 18:21:12]

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

[commit-bot: I haz the power, 2016-03-10 18:21:13]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[pdr., 2016-03-10 20:58:07]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2016-03-10 20:58:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1758413002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1758413002/1

[commit-bot: I haz the power, 2016-03-10 22:11:57]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-10 22:13:19]

Description was changed from

==========
Do not pass scrollPosition to updateScrollbars at every callsite

More followup cleanup to the scroll position code. This is a minor
refactoring to push the scroll position double concent into
updateScrollbars since it does not need to be plumbed at every callsite.
scrollPositionDouble has some subtlety (see comment above it) which is
easy to mess up.
==========

to

==========
Do not pass scrollPosition to updateScrollbars at every callsite

More followup cleanup to the scroll position code. This is a minor
refactoring to push the scroll position double concent into
updateScrollbars since it does not need to be plumbed at every callsite.
scrollPositionDouble has some subtlety (see comment above it) which is
easy to mess up.

Committed: https://crrev.com/d12c2b3ab25398ae385564c7506f0e4b598169eb
Cr-Commit-Position: refs/heads/master@{#380478}
==========

[commit-bot: I haz the power, 2016-03-10 22:13:20]

Patchset 1 (id:??) landed as
https://crrev.com/d12c2b3ab25398ae385564c7506f0e4b598169eb
Cr-Commit-Position: refs/heads/master@{#380478}