Replace homebrew scrollbar in Files app with that of Blink.

Replace homebrew scrollbar in Files app with that of Blink.

When overlay scrollbar is enabled in Chrome, there is animation to shrink and fade out the scrollbar thumb when inactive. However, scroll bars that aren't composited will not be animated
(see Issue 671644). It would make the scroll bar always thick and simply disappear without fading out after inactive.
As a workaround, this change makes the 2 boxes have solid background and contain:paint so as to make sure the scrollers are composited.

This changelist is basically same as Issue 2554433002, which was revereted once.
This changelist also contains a fix for Issue 697057 in addition.

BUG=666589, 697057
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

[yamaguchi, 2017-03-03 04:16:39]

Description was changed from

==========
Replace homebrew scrollbar in Files app with that of Blink.

When overlay scrollbar is enabled in Chrome, there is animation to shrink and
fade out the scrollbar thumb when inactive. However, scroll bars that aren't
composited will not be animated
(see Issue 671644). It would make the scroll bar always thick and simply
disappear without fading out after inactive.
As a workaround, this change makes the 2 boxes have solid background and
contain:paint so as to make sure the scrollers are composited.

This changelist is basically same as Issue 2554433002, which was revereted once.
This changelist also contains a fix for Issue 697057 in addition.

BUG=666589,697057
==========

to

==========
Replace homebrew scrollbar in Files app with that of Blink.

When overlay scrollbar is enabled in Chrome, there is animation to shrink and
fade out the scrollbar thumb when inactive. However, scroll bars that aren't
composited will not be animated
(see Issue 671644). It would make the scroll bar always thick and simply
disappear without fading out after inactive.
As a workaround, this change makes the 2 boxes have solid background and
contain:paint so as to make sure the scrollers are composited.

This changelist is basically same as Issue 2554433002, which was revereted once.
This changelist also contains a fix for Issue 697057 in addition.

BUG=666589,697057
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[yamaguchi, 2017-03-28 14:25:05]

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

[commit-bot: I haz the power, 2017-03-28 14:25:34]

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

[yamaguchi, 2017-03-28 14:33:49]

yamaguchi@chromium.org changed reviewers:
+ fukino@chromium.org

[yamaguchi, 2017-03-28 14:33:50]

ptal

https://codereview.chromium.org/2724303003/diff/40001/ui/file_manager/file_ma...
File ui/file_manager/file_manager/foreground/css/file_manager.css (right):

https://codereview.chromium.org/2724303003/diff/40001/ui/file_manager/file_ma...
ui/file_manager/file_manager/foreground/css/file_manager.css:132: /*
TODO(yamaguchi): Remove these when crbug.com/671644 is resolved. */
Now we can remove this hack (which we needed just for making the overlay
scrollbar collapsible) because crrev/2758733003 was checked in.

[commit-bot: I haz the power, 2017-03-28 14:51:44]

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

[commit-bot: I haz the power, 2017-03-28 14:51:44]

Dry run: This issue passed the CQ dry run.

[fukino, 2017-03-29 01:41:31]

> This changelist is basically same as Issue 2554433002, which was revereted
once.
If possible, it would be great if you upload the reland patch as PS1 and add
additional changes as PS2.

> This changelist also contains a fix for Issue 697057 in addition.
Could you do it in a separate CL?
(It's https://codereview.chromium.org/2745593003/, isn't it?)

[yamaguchi, 2017-03-29 03:44:57]

On 2017/03/29 01:41:31, fukino wrote:
> > This changelist is basically same as Issue 2554433002, which was revereted
> once.
> If possible, it would be great if you upload the reland patch as PS1 and add
> additional changes as PS2.
> 
> > This changelist also contains a fix for Issue 697057 in addition.
> Could you do it in a separate CL?
> (It's https://codereview.chromium.org/2745593003/, isn't it?)

Created another Issue using a reland patch to facilitate review.
https://codereview.chromium.org/2782063002
I will drop this changelist.