Use overlay scrollbars for the cros system menu.

Use overlay scrollbars for the cros system menu.

These scrollbars are already used in the message center.

These scrollbars are additionally applied to the CrOS system menu tray detail
views (e.g. the list of wifi networks). At a later date, we can enhance
these scrollbars (make sure they match Sebastien's specs) and
generalize their use to more ChromeOS surfaces. Since it affects layout,
we have to carefully make sure not to break anything.

Depends on https://codereview.chromium.org/2480763005/

BUG=657159
Committed: https://crrev.com/744d9b779a63b34bce18c0f2ef9206d4565af1ce
Cr-Commit-Position: refs/heads/master@{#430619}

[Evan Stade, 2016-11-05 01:05:39]

Description was changed from

==========
Use pseudo-overlay scrollbars for the cros system menu.

There is no space reserved for the track, and the track is transparent,
but the thumb is always present.

These scrollbars are only applied to the CrOS system menu tray detail
views (e.g. the list of wifi networks). At a later date, we can enhance
these scrollbars (add more cool overlay features like auto-hide) and
generalize their use to more ChromeOS surfaces. Since it affects layout,
we have to carefully make sure not to break anything

BUG=657159
==========

to

==========
Use pseudo-overlay scrollbars for the cros system menu.

There is no space reserved for the track, and the track is transparent,
but the thumb is always present.

These scrollbars are only applied to the CrOS system menu tray detail
views (e.g. the list of wifi networks). At a later date, we can enhance
these scrollbars (add more cool overlay features like auto-hide) and
generalize their use to more ChromeOS surfaces. Since it affects layout,
we have to carefully make sure not to break anything.

Depends on https://codereview.chromium.org/2480763005/

BUG=657159
==========

[Evan Stade, 2016-11-05 01:05:40]

estade@chromium.org changed reviewers:
+ tdanderson@chromium.org

[Evan Stade, 2016-11-05 01:44:13]

Description was changed from

==========
Use pseudo-overlay scrollbars for the cros system menu.

There is no space reserved for the track, and the track is transparent,
but the thumb is always present.

These scrollbars are only applied to the CrOS system menu tray detail
views (e.g. the list of wifi networks). At a later date, we can enhance
these scrollbars (add more cool overlay features like auto-hide) and
generalize their use to more ChromeOS surfaces. Since it affects layout,
we have to carefully make sure not to break anything.

Depends on https://codereview.chromium.org/2480763005/

BUG=657159
==========

to

==========
Use pseudo-overlay scrollbars for the cros system menu.

There is no space reserved for the track, and the track is transparent,
but the thumb is always present (updating thumb painting is a TODO).

These scrollbars are only applied to the CrOS system menu tray detail
views (e.g. the list of wifi networks). At a later date, we can enhance
these scrollbars (add more cool overlay features like auto-hide) and
generalize their use to more ChromeOS surfaces. Since it affects layout,
we have to carefully make sure not to break anything.

Depends on https://codereview.chromium.org/2480763005/

BUG=657159
==========

[tdanderson, 2016-11-07 22:33:21]

LGTM

https://codereview.chromium.org/2478273003/diff/40001/ash/common/system/tray/...
File ash/common/system/tray/fixed_sized_scroll_view.cc (right):

https://codereview.chromium.org/2478273003/diff/40001/ash/common/system/tray/...
ash/common/system/tray/fixed_sized_scroll_view.cc:20: // for the track, and the
track is transparent, but the thumb is always present.
Can you link to / file a follow up bug (and put the crbug number here) to
animate the thumb out like in web contents?

[Evan Stade, 2016-11-07 23:37:09]

Description was changed from

==========
Use pseudo-overlay scrollbars for the cros system menu.

There is no space reserved for the track, and the track is transparent,
but the thumb is always present (updating thumb painting is a TODO).

These scrollbars are only applied to the CrOS system menu tray detail
views (e.g. the list of wifi networks). At a later date, we can enhance
these scrollbars (add more cool overlay features like auto-hide) and
generalize their use to more ChromeOS surfaces. Since it affects layout,
we have to carefully make sure not to break anything.

Depends on https://codereview.chromium.org/2480763005/

BUG=657159
==========

to

==========
Use overlay scrollbars for the cros system menu.

These scrollbars are already used in the message center.

These scrollbars are additionally applied to the CrOS system menu tray detail
views (e.g. the list of wifi networks). At a later date, we can enhance
these scrollbars (make sure they match Sebastien's specs) and
generalize their use to more ChromeOS surfaces. Since it affects layout,
we have to carefully make sure not to break anything.

Depends on https://codereview.chromium.org/2480763005/

BUG=657159
==========

[Evan Stade, 2016-11-08 00:37:00]

sorry for churn, please re-review. Everything in ime seems to work as it should
except for an orthogonal breakage that I've tracked down to HoverHighlightView
badness.

[tdanderson, 2016-11-08 01:23:39]

Patch Set 4 LGTM

[Evan Stade, 2016-11-08 01:26:46]

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

[Evan Stade, 2016-11-08 01:26:57]

The CQ bit was unchecked by estade@chromium.org

[Evan Stade, 2016-11-08 01:27:20]

estade@chromium.org changed reviewers:
+ sadrul@chromium.org

[Evan Stade, 2016-11-08 01:27:21]

+sadrul for ui/views

[sadrul, 2016-11-08 01:43:24]

lgtm

[Evan Stade, 2016-11-08 01:45:05]

The CQ bit was checked by estade@chromium.org

[Evan Stade, 2016-11-08 01:45:05]

The patchset sent to the CQ was uploaded after l-g-t-m from
tdanderson@chromium.org
Link to the patchset: https://codereview.chromium.org/2478273003/#ps80001
(title: "find missing change")

[commit-bot: I haz the power, 2016-11-08 01:45:37]

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, 2016-11-08 01:48:59]

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

[commit-bot: I haz the power, 2016-11-08 01:49:00]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[Evan Stade, 2016-11-08 14:44:02]

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

[commit-bot: I haz the power, 2016-11-08 14:44:14]

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, 2016-11-08 14:55:20]

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

[commit-bot: I haz the power, 2016-11-08 14:55:21]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[Evan Stade, 2016-11-08 15:07:12]

The CQ bit was checked by estade@chromium.org

[Evan Stade, 2016-11-08 15:07:12]

The patchset sent to the CQ was uploaded after l-g-t-m from
tdanderson@chromium.org, sadrul@chromium.org
Link to the patchset: https://codereview.chromium.org/2478273003/#ps100001
(title: "rebase")

[commit-bot: I haz the power, 2016-11-08 15:07:30]

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, 2016-11-08 15:36:29]

Description was changed from

==========
Use overlay scrollbars for the cros system menu.

These scrollbars are already used in the message center.

These scrollbars are additionally applied to the CrOS system menu tray detail
views (e.g. the list of wifi networks). At a later date, we can enhance
these scrollbars (make sure they match Sebastien's specs) and
generalize their use to more ChromeOS surfaces. Since it affects layout,
we have to carefully make sure not to break anything.

Depends on https://codereview.chromium.org/2480763005/

BUG=657159
==========

to

==========
Use overlay scrollbars for the cros system menu.

These scrollbars are already used in the message center.

These scrollbars are additionally applied to the CrOS system menu tray detail
views (e.g. the list of wifi networks). At a later date, we can enhance
these scrollbars (make sure they match Sebastien's specs) and
generalize their use to more ChromeOS surfaces. Since it affects layout,
we have to carefully make sure not to break anything.

Depends on https://codereview.chromium.org/2480763005/

BUG=657159
==========

[commit-bot: I haz the power, 2016-11-08 15:36:30]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-11-08 15:41:56]

Description was changed from

==========
Use overlay scrollbars for the cros system menu.

These scrollbars are already used in the message center.

These scrollbars are additionally applied to the CrOS system menu tray detail
views (e.g. the list of wifi networks). At a later date, we can enhance
these scrollbars (make sure they match Sebastien's specs) and
generalize their use to more ChromeOS surfaces. Since it affects layout,
we have to carefully make sure not to break anything.

Depends on https://codereview.chromium.org/2480763005/

BUG=657159
==========

to

==========
Use overlay scrollbars for the cros system menu.

These scrollbars are already used in the message center.

These scrollbars are additionally applied to the CrOS system menu tray detail
views (e.g. the list of wifi networks). At a later date, we can enhance
these scrollbars (make sure they match Sebastien's specs) and
generalize their use to more ChromeOS surfaces. Since it affects layout,
we have to carefully make sure not to break anything.

Depends on https://codereview.chromium.org/2480763005/

BUG=657159

Committed: https://crrev.com/744d9b779a63b34bce18c0f2ef9206d4565af1ce
Cr-Commit-Position: refs/heads/master@{#430619}
==========

[commit-bot: I haz the power, 2016-11-08 15:41:58]

Patchset 6 (id:??) landed as
https://crrev.com/744d9b779a63b34bce18c0f2ef9206d4565af1ce
Cr-Commit-Position: refs/heads/master@{#430619}