Kill ScrollbarModesCalculationStrategy.

Kill ScrollbarModesCalculationStrategy.

This was added in http://crrev.com/9de09a32, but I don't think it makes sense
anymore.  Its purpose is to include child frames in FV::scrollable_areas_ even
if their scrollbars have been suppressed by FV::SetCanHaveScrollbars(false).
But such frames are not scrollable by user input (see FV::UserInputScrollable
which examines the ScrollbarMode values), so there is no need for them to be in
FV::scrollable_areas_.

BUG=724309
Review-Url: https://codereview.chromium.org/2894733002
Cr-Commit-Position: refs/heads/master@{#473100}
Committed: https://chromium.googlesource.com/chromium/src/+/6130f3dd3af923ec847c32372351495d5b8878fd

[skobes, 2017-05-18 23:21:03]

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

[commit-bot: I haz the power, 2017-05-18 23:22:43]

Dry run: CQ is trying da patch.

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

[skobes, 2017-05-19 00:49:25]

Description was changed from

==========
Ignore ScrollbarModesCalculationStrategy.

BUG=724309
==========

to

==========
Kill ScrollbarModesCalculationStrategy.

This was added in http://crrev.com/9de09a32, but I don't think it makes sense
anymore.  Its purpose is to include child frames in FV::scrollable_areas_ even
if their scrollbars have been suppressed by FV::SetCanHaveScrollbars(false).
But such frames are not scrollable by user input (see FV::UserInputScrollable
which examines the ScrollbarMode values), so there is no need for them to be in
FV::scrollable_areas_.

BUG=724309
==========

[skobes, 2017-05-19 00:49:59]

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

[commit-bot: I haz the power, 2017-05-19 00:51:03]

Dry run: CQ is trying da patch.

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

[skobes, 2017-05-19 00:51:52]

Patchset #1 (id:1) has been deleted

[skobes, 2017-05-19 00:52:06]

skobes@chromium.org changed reviewers:
+ szager@chromium.org

[szager1, 2017-05-19 03:29:32]

lgtm

[skobes, 2017-05-19 03:34:27]

The CQ bit was unchecked by skobes@chromium.org

[skobes, 2017-05-19 03:34:30]

The CQ bit was checked by skobes@chromium.org

[commit-bot: I haz the power, 2017-05-19 03:34:56]

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-05-19 05:46:31]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495164870342910,
"parent_rev": "97cf10ffd89f897b67f489c4b072886e186c9944", "commit_rev":
"6130f3dd3af923ec847c32372351495d5b8878fd"}

[commit-bot: I haz the power, 2017-05-19 05:46:42]

Description was changed from

==========
Kill ScrollbarModesCalculationStrategy.

This was added in http://crrev.com/9de09a32, but I don't think it makes sense
anymore.  Its purpose is to include child frames in FV::scrollable_areas_ even
if their scrollbars have been suppressed by FV::SetCanHaveScrollbars(false).
But such frames are not scrollable by user input (see FV::UserInputScrollable
which examines the ScrollbarMode values), so there is no need for them to be in
FV::scrollable_areas_.

BUG=724309
==========

to

==========
Kill ScrollbarModesCalculationStrategy.

This was added in http://crrev.com/9de09a32, but I don't think it makes sense
anymore.  Its purpose is to include child frames in FV::scrollable_areas_ even
if their scrollbars have been suppressed by FV::SetCanHaveScrollbars(false).
But such frames are not scrollable by user input (see FV::UserInputScrollable
which examines the ScrollbarMode values), so there is no need for them to be in
FV::scrollable_areas_.

BUG=724309

Review-Url: https://codereview.chromium.org/2894733002
Cr-Commit-Position: refs/heads/master@{#473100}
Committed:
https://chromium.googlesource.com/chromium/src/+/6130f3dd3af923ec847c32372351...
==========

[commit-bot: I haz the power, 2017-05-19 05:46:43]

Committed patchset #1 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/6130f3dd3af923ec847c32372351...