Add auto scrollbars if content would be visible without them.

Add auto scrollbars if content would be visible without them.

The check to avoid adding auto scrollbars if the contents aren't
visible should consider whether the contents would be visible in the
absence of a scrollbar.  It's a weird edge case, because adding the
scrollbar will hide the content, but it's the only reasonable thing to
do.

BUG=690115
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2759213002
Cr-Commit-Position: refs/heads/master@{#458348}
Committed: https://chromium.googlesource.com/chromium/src/+/8c762e79eef5c9a6d7bce492ec42e8870ab21f8e

[szager1, 2017-03-20 18:03:53]

Description was changed from

==========
Add auto scrollbars if content would be visible without them.

The check to avoid adding auto scrollbars if the contents aren't
visible should consider whether the contents would be visible in the
absence of a scrollbar.  It's a weird edge case, because adding the
scrollbar will hide the content, but it's the only reasonable thing to
do.

BUG=690115
==========

to

==========
Add auto scrollbars if content would be visible without them.

The check to avoid adding auto scrollbars if the contents aren't
visible should consider whether the contents would be visible in the
absence of a scrollbar.  It's a weird edge case, because adding the
scrollbar will hide the content, but it's the only reasonable thing to
do.

BUG=690115
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[szager1, 2017-03-20 18:04:38]

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

[commit-bot: I haz the power, 2017-03-20 18:05:21]

Dry run: 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-03-20 21:02:02]

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

[commit-bot: I haz the power, 2017-03-20 21:02:03]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[szager1, 2017-03-21 03:09:58]

Description was changed from

==========
Add auto scrollbars if content would be visible without them.

The check to avoid adding auto scrollbars if the contents aren't
visible should consider whether the contents would be visible in the
absence of a scrollbar.  It's a weird edge case, because adding the
scrollbar will hide the content, but it's the only reasonable thing to
do.

BUG=690115
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Add auto scrollbars if content would be visible without them.

The check to avoid adding auto scrollbars if the contents aren't
visible should consider whether the contents would be visible in the
absence of a scrollbar.  It's a weird edge case, because adding the
scrollbar will hide the content, but it's the only reasonable thing to
do.

BUG=690115
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[szager1, 2017-03-21 03:09:58]

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

[szager1, 2017-03-21 03:10:17]

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

[commit-bot: I haz the power, 2017-03-21 03:10:50]

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

[skobes, 2017-03-21 03:44:23]

lgtm

[commit-bot: I haz the power, 2017-03-21 05:25:54]

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

[commit-bot: I haz the power, 2017-03-21 05:25:55]

Dry run: This issue passed the CQ dry run.

[szager1, 2017-03-21 07:47:51]

The CQ bit was checked by szager@chromium.org

[commit-bot: I haz the power, 2017-03-21 07:48:08]

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-03-21 07:51:38]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490082471205610,
"parent_rev": "002f07fbe7776175f47bf49a284f673e26c409b3", "commit_rev":
"8c762e79eef5c9a6d7bce492ec42e8870ab21f8e"}

[commit-bot: I haz the power, 2017-03-21 07:52:18]

Description was changed from

==========
Add auto scrollbars if content would be visible without them.

The check to avoid adding auto scrollbars if the contents aren't
visible should consider whether the contents would be visible in the
absence of a scrollbar.  It's a weird edge case, because adding the
scrollbar will hide the content, but it's the only reasonable thing to
do.

BUG=690115
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Add auto scrollbars if content would be visible without them.

The check to avoid adding auto scrollbars if the contents aren't
visible should consider whether the contents would be visible in the
absence of a scrollbar.  It's a weird edge case, because adding the
scrollbar will hide the content, but it's the only reasonable thing to
do.

BUG=690115
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2759213002
Cr-Commit-Position: refs/heads/master@{#458348}
Committed:
https://chromium.googlesource.com/chromium/src/+/8c762e79eef5c9a6d7bce492ec42...
==========

[commit-bot: I haz the power, 2017-03-21 07:52:19]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/8c762e79eef5c9a6d7bce492ec42...